Re: Should we remove openid from Sling's launchpad?

2014-09-26 Thread Carsten Ziegeler
Not sure if we have to touch auth for Oak in general, I though the basics should work (but I'm not 100% sure). One more reason to switch to Oak once SLING-7 is out Carsten 2014-09-25 16:52 GMT+02:00 Oliver Lietz apa...@oliverlietz.de: On Wednesday 24 September 2014 16:03:28 Antonio Sanso

Re: Should we remove openid from Sling's launchpad?

2014-09-25 Thread Oliver Lietz
On Wednesday 24 September 2014 16:03:28 Antonio Sanso wrote: On Sep 24, 2014, at 1:51 PM, Oliver Lietz apa...@oliverlietz.de wrote: On Wednesday 24 September 2014 11:13:59 Carsten Ziegeler wrote: Our launchpad currently contains openid support - which is old/not recommended to be used

Should we remove openid from Sling's launchpad?

2014-09-24 Thread Carsten Ziegeler
Our launchpad currently contains openid support - which is old/not recommended to be used anymore. And maybe the implementation is also broken... So I think we should not have this in Sling-7 WDYT? Carsten -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Robert Munteanu
+1 for removing the openid bundle Robert On Wed, Sep 24, 2014 at 12:13 PM, Carsten Ziegeler cziege...@apache.org wrote: Our launchpad currently contains openid support - which is old/not recommended to be used anymore. And maybe the implementation is also broken... So I think we should not

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Oliver Lietz
On Wednesday 24 September 2014 11:13:59 Carsten Ziegeler wrote: Our launchpad currently contains openid support - which is old/not recommended to be used anymore. And maybe the implementation is also broken... So I think we should not have this in Sling-7 WDYT? +1 for removing broken

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Carsten Ziegeler
It seems we have to also remove the auth selector model which has hard coded stuff for open id in it. Not sure what it really does Carsten 2014-09-24 13:51 GMT+02:00 Oliver Lietz apa...@oliverlietz.de: On Wednesday 24 September 2014 11:13:59 Carsten Ziegeler wrote: Our launchpad

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Carsten Ziegeler
If no one objects I'll remove the auth selector as well - its really only usable with the openid stuff. Carsten 2014-09-24 15:04 GMT+02:00 Carsten Ziegeler cziege...@apache.org: It seems we have to also remove the auth selector model which has hard coded stuff for open id in it. Not sure what

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Antonio Sanso
On Sep 24, 2014, at 1:51 PM, Oliver Lietz apa...@oliverlietz.de wrote: On Wednesday 24 September 2014 11:13:59 Carsten Ziegeler wrote: Our launchpad currently contains openid support - which is old/not recommended to be used anymore. And maybe the implementation is also broken... So I

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Oliver Lietz
On Wednesday 24 September 2014 15:04:50 Carsten Ziegeler wrote: It seems we have to also remove the auth selector model which has hard coded stuff for open id in it. Not sure what it really does AFAIR it's for switching auth modules from a user's pov (selecting). It depends on Form and

Re: Should we remove openid from Sling's launchpad?

2014-09-24 Thread Carsten Ziegeler
Right, so it's use is really limited to the openid stuff - so I removed it from launchpad and moved the module to the contrib section Regards Carsten 2014-09-24 17:32 GMT+02:00 Oliver Lietz apa...@oliverlietz.de: On Wednesday 24 September 2014 15:04:50 Carsten Ziegeler wrote: It seems we