Project

General

Profile

Actions
Copy link

feature request #3133

closed

security: preparation and first steps for Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority

Added by Andreas Kohlbecker over 11 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Priority11
Assignee:
Andreas Kohlbecker
Category:
cdmlib
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Severity:
normal
Tags:
security

Description

There is no relation in the api between Role, CdmAuthority and GrantedAuthorityImpl they are connected only via the string representations. But all are GrantedAuthorities and thus should implement the interface and should provide conversion functionality which allows to convert from Role and CdmAuthority to GrantedAuthorityImpl and vice versa.

old subject:

security: merge Role enum and CdmAuthority into GrantedAuthorityImpl

old description:

There is now an enum for roles: @GrantedAuthorityImpl.Role@.

It would make more sense however to have all roles as final static instances of GrantedAuthorityImpl and it would also make sense to merge eu.etaxonomy.cdm.persistence.hibernate.permission.CdmAuthority into GrantedAuthorityImpl => 7099

Related issues

Copied to EDIT - task #7099: Make CdmAuthority a persistable classClosedAndreas Müller

Actions
Actions
Copy link
 #1

Updated by Andreas Kohlbecker over 11 years ago

  • Status changed from New to In Progress

hi cherian I will start working on this today

Actions
Copy link
 #2

Updated by Andreas Kohlbecker over 11 years ago

  • Subject changed from security: merge Role enum and CdmAuthority into GrantedAuthorityImpl to security: relate Role, CdmAuthority and GrantedAuthorityImpl to each other

completely merging all classes together seems not to make fully sense. So we decided to provide some conversion functionality which allows to convert from Role and CdmAuthority to GrantedAuthorityImpl and vice versa. Role and CdmAuthority should be in the cdmlib-persistence package.

I am also changing the subject of this ticket accordingly.

Actions
Copy link
 #3

Updated by Andreas Kohlbecker over 11 years ago

  • Subject changed from security: relate Role, CdmAuthority and GrantedAuthorityImpl to each other to security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority
Actions
Copy link
 #4

Updated by Andreas Kohlbecker over 10 years ago

  • Target version changed from cdmlib RELEASE 3.3.0 to cdmlib RELEASE 3.3.1

moving unresolved tickets to next release milestone due release of version 3.3

Actions
Copy link
 #5

Updated by Andreas Kohlbecker about 9 years ago

  • Target version changed from cdmlib RELEASE 3.5.0 to cdmlib RELEASE 3.5.1

moving tickets to next milestone

Actions
Copy link
 #6

Updated by Andreas Müller almost 9 years ago

  • Target version changed from cdmlib RELEASE 3.5.1 to cdmlib RELEASE 3.5.2

move open 3.5.1 tickets to next milestone after release

Actions
Copy link
 #7

Updated by Katja Luther almost 9 years ago

  • Keywords set to security,
  • Target version changed from cdmlib RELEASE 3.5.2 to cdmlib - Reviewed Next Major Release
  • Priority changed from Priority08 to Priority11
Actions
Copy link
 #8

Updated by Andreas Müller over 6 years ago

  • Copied to task #7099: Make CdmAuthority a persistable class added
Actions
Copy link
 #9

Updated by Andreas Müller over 6 years ago

  • Description updated (diff)

Are there still open issues?

Actions
Copy link
 #10

Updated by Andreas Kohlbecker over 6 years ago

  • Subject changed from security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority to security: preparation and first steps for Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority
  • Status changed from In Progress to Closed
  • Target version deleted (Reviewed Next Major Release)

all requirements defined in this issue are now covered by #7099, so we can close this ticked

Actions
Copy link
 #11

Updated by Andreas Kohlbecker over 6 years ago

  • Private changed from Yes to No
Actions
Copy link

Also available in: Atom PDF