Project

General

Profile

feature request #3133

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

Added by Andreas Kohlbecker almost 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Priority11
Category:
cdmlib
Target version:
-
Start date:
10/05/2012
Due date:
% Done:

0%

Severity:
normal
Tags:

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 - bug #7099: Make CdmAuthority a persistable class Resolved 11/30/2017

Associated revisions

Revision f4cf248d (diff)
Added by Andreas Kohlbecker almost 7 years ago

fixing #3133 (security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority)

History

#1 Updated by Andreas Kohlbecker almost 7 years ago

  • Status changed from New to In Progress

hi cherian I will start working on this today

#2 Updated by Andreas Kohlbecker almost 7 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.

#3 Updated by Andreas Kohlbecker almost 7 years ago

  • Subject changed from security: relate Role, CdmAuthority and GrantedAuthorityImpl to each other to security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority

#4 Updated by Andreas Kohlbecker over 5 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

#5 Updated by Andreas Kohlbecker over 4 years ago

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

moving tickets to next milestone

#6 Updated by Andreas Müller over 4 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

#7 Updated by Katja Luther over 4 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

#8 Updated by Andreas Müller almost 2 years ago

  • Copied to bug #7099: Make CdmAuthority a persistable class added

#9 Updated by Andreas Müller almost 2 years ago

  • Description updated (diff)

Are there still open issues?

#10 Updated by Andreas Kohlbecker almost 2 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

#11 Updated by Andreas Kohlbecker almost 2 years ago

  • Private changed from Yes to No

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)