[security-dev] PicketLink IDM JPA Identity Store

[Bruno Oliveira]

Hi guys, which line should I look? From my perspective this line https://github.com/picketlink/picketlink-idm-restored/blob/master/impl/src/test/java/org/picketlink/test/idm/internal/mgr/DefaultJPAIdentityManagerTestCase.java#L71 could be refactored to https://github.com/picketlink/picketlink-idm-restored/blob/6e63bc583fa2fab5976cac7449c39be447b03536/impl/src/test/java/org/picketlink/test/idm/internal/jpa/JPAUserTestCase.java#L94

We're able to implement our own User entity, might be interesting to add getPassword/setPassword methods, in this way this line https://github.com/picketlink/picketlink-idm-restored/blob/master/impl/src/test/java/org/picketlink/test/idm/internal/mgr/DefaultJPAIdentityManagerTestCase.java#L121 won't be needed. Just doing would be enough:

public class MyCustomUser implements User {
….
}

myCustomUser.setPassword("…"); //think about the registration process, we don't need DTOs just to pass passwords between entities.

im.createUser(myCustomUser);


About this line https://github.com/picketlink/picketlink-idm-restored/blob/master/impl/src/test/java/org/picketlink/test/idm/internal/mgr/DefaultJPAIdentityManagerTestCase.java#L138 I'll refactor it and send a PR.

Currently we don't have groups on AeroGear, so our method is something like:

im.grantRole(adminRole, user, null);  

I'll refactor it to allow users without a group. Wdyt?


--  
"The measure of a man is what he does with power" - Plato
-
@abstractj
-
Volenti Nihil Difficile



On Monday, October 8, 2012 at 1:30 PM, Anil Saldhana wrote:

> I want to offer continued discussion on the JPA implementation in the  
> IDM project.
>  
> The work that Pedro did is restored here in the following workspace:
> https://github.com/picketlink/picketlink-idm-restored
>  
> A testcase that is useful for JPA implementation in IDM is:  
> https://github.com/picketlink/picketlink-idm-restored/blob/master/impl/src/test/java/org/picketlink/test/idm/internal/mgr/DefaultJPAIdentityManagerTestCase.java
>  
> It is the exact mirror of the LDAP implementation:  
> https://github.com/picketlink/picketlink-idm-restored/blob/master/impl/src/test/java/org/picketlink/test/idm/internal/mgr/DefaultLDAPIdentityManagerTestCase.java
>  
> These two implementations have very minimal user configuration.
>  
> The challenge is when users bring in complex database schemas and LDAP  
> DITs into operation. But the goal of balancing complexity with  
> usability is a tough one.
>  
> On 09/06/2012 10:13 AM, Anil Saldhana wrote:
> > Similar challenges exist for LDAP bindings also, since user LDAP DITs
> > may be different. But we have to balance complexity with usability. :)
> >  
> > On 09/06/2012 07:37 AM, Pedro Igor Silva wrote:
> > > Ok. I'll take a look how he took care of that.
> > >  
> > > Regards.
> > > Pedro Igor
> > >  
> > > ----- Original Message -----
> > > From: "Anil Saldhana" <Anil.Saldhana at redhat.com (mailto:Anil.Saldhana at redhat.com)>
> > > To: security-dev at lists.jboss.org (mailto:security-dev at lists.jboss.org)
> > > Sent: Wednesday, September 5, 2012 6:52:35 PM
> > > Subject: [security-dev] PicketLink IDM JPA Identity Store
> > >  
> > > Pedro,
> > > Shane just referred me to the following:
> > >  
> > > https://github.com/seam/security/blob/develop/impl/src/main/java/org/jboss/seam/security/management/picketlink/JpaIdentityStore.java
> > >  
> > > Can you adapt your work to incorporate all facets of this Seam work?
> > > Shane says users have varying db schema structures and the JPA
> > > implementation in seam3 took care of the nuances.
> > >  
> > > Regards,
> > > Anil
> > >  
> >  
> >  
>  
> _______________________________________________
> security-dev mailing list
> security-dev at lists.jboss.org (mailto:security-dev at lists.jboss.org)
> https://lists.jboss.org/mailman/listinfo/security-dev
>  
>  


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/security-dev/attachments/20121008/743a3604/attachment.html