Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3
Shane Hathaway wrote:
Log message for revision 97183: New library for OpenID auth in Zope 3
Changed: A zope.app.openidconsumer/
One question: why is this in zope.app? I think there's a consensus we're trying to pull as much from zope.app as possible. Is this going to provide a ZMI UI at all? If not, I'd suggest putting it in zope.* Regards, Martijn
Martijn Faassen wrote:
One question: why is this in zope.app? I think there's a consensus we're trying to pull as much from zope.app as possible.
Is this going to provide a ZMI UI at all? If not, I'd suggest putting it in zope.*
I admit I'm being lazy here. It seems like zope.app is a dumping ground for packages with muddy dependencies. I didn't want to work out the dependency list yet. :-) I do have some justification though: it depends on IAuthentication, which is in zope.app.security. Still, I suspect IAuthentication needs to be moved out of zope.app.security. Shane
On Feb 24, 2009, at 3:17 PM, Shane Hathaway wrote:
Martijn Faassen wrote:
One question: why is this in zope.app? I think there's a consensus we're trying to pull as much from zope.app as possible.
Is this going to provide a ZMI UI at all? If not, I'd suggest putting it in zope.*
I admit I'm being lazy here. It seems like zope.app is a dumping ground for packages with muddy dependencies. I didn't want to work out the dependency list yet. :-)
I do have some justification though: it depends on IAuthentication, which is in zope.app.security. Still, I suspect IAuthentication needs to be moved out of zope.app.security.
I agree that it shouldn't go in zope.app. I believe I suggested putting this in zc.openid, although I'm fine with zope.openid. Placement is more a function of project or originating organization, rather than dependencies. The top-level package is determined by the project/organization. Normally, there should be only 2 levels of packages. (I screwed up with the zc.recipes package, for example.) Jim -- Jim Fulton Zope Corporation
2009/2/24 Jim Fulton <jim@zope.com>:
I agree that it shouldn't go in zope.app. I believe I suggested putting this in zc.openid, although I'm fine with zope.openid.
Why zc? I thought it's only for packages coming from the zope corporation. Or does Shane works for ZC? :) -- WBR, Dan Korostelev
Dan Korostelev wrote:
2009/2/24 Jim Fulton <jim@zope.com>:
I agree that it shouldn't go in zope.app. I believe I suggested putting this in zc.openid, although I'm fine with zope.openid.
Why zc? I thought it's only for packages coming from the zope corporation. Or does Shane works for ZC? :)
This is for a ZC contract. Shane
2009/2/24 Shane Hathaway <shane@hathawaymix.org>:
Dan Korostelev wrote:
2009/2/24 Jim Fulton <jim@zope.com>:
I agree that it shouldn't go in zope.app. I believe I suggested putting this in zc.openid, although I'm fine with zope.openid.
Why zc? I thought it's only for packages coming from the zope corporation. Or does Shane works for ZC? :)
This is for a ZC contract.
Ah, then zc namespace if fine of course! :) -- WBR, Dan Korostelev
Hi there, I hope in fact zope.app.* will soon become a dumping ground for deprecated packages providing legacy ZMI support. Of course that will need the consensus that the ZMI *is* legacy software. I think do we already have a consensus that packages that provide other useful functionality shouldn't be providing ZMI support within the same package. Regards, Martijn
Hi Shane
Betreff: Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3
Martijn Faassen wrote:
One question: why is this in zope.app? I think there's a consensus we're trying to pull as much from zope.app as possible.
Is this going to provide a ZMI UI at all? If not, I'd suggest putting it in zope.*
I admit I'm being lazy here. It seems like zope.app is a dumping ground for packages with muddy dependencies. I didn't want to work out the dependency list yet. :-)
I do have some justification though: it depends on IAuthentication, which is in zope.app.security. Still, I suspect IAuthentication needs to be moved out of zope.app.security.
just a note; I have still a refactoring pending which which I proposed month ago. This will move the IPasswordManager out of zope.app.authentication. Regards Roger Ineichen
Shane
_______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
participants (5)
-
Dan Korostelev -
Jim Fulton -
Martijn Faassen -
Roger Ineichen -
Shane Hathaway