Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-25 Thread Roger Ineichen
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 )
 

___
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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Martijn Faassen
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




___
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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Jim Fulton

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


___
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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Shane Hathaway
Jim Fulton wrote:
 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.

I'll rename it to zc.openid.

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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Dan Korostelev
2009/2/24 Jim Fulton j...@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
___
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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Shane Hathaway
Dan Korostelev wrote:
 2009/2/24 Jim Fulton j...@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

___
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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Dan Korostelev
2009/2/24 Shane Hathaway sh...@hathawaymix.org:
 Dan Korostelev wrote:

 2009/2/24 Jim Fulton j...@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
___
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 )


Re: [Zope-dev] SVN: zope.app.openidconsumer/ New library for OpenID auth in Zope 3

2009-02-24 Thread Martijn Faassen
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
___
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 )