Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-21 Thread Chris Withers
Florent Guillaume wrote: Ah, okay, I thought that's what you meant, but I hoped it wasn't. The fact that you expect this to work is a bug in Zope's security machinery, IMHO, but sadly only IMHO it appears. Huh? That's fundamental to Zope's security model. As I said, I appear to be the only person

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-18 Thread Florent Guillaume
Chris Withers [EMAIL PROTECTED] wrote: A, B and C are folders nested in each other i.e. A/B/C. A user does not have access to A and B but he does have access to C. If getObject uses restrictedTraverse it returns None immediately when traversing A, even though the user is allowed to access

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-16 Thread Chris Withers
Roché Compaan wrote: I don't get why you're not getting it :-) A, B and C are folders nested in each other i.e. A/B/C. A user does not have access to A and B but he does have access to C. If getObject uses restrictedTraverse it returns None immediately when traversing A, even though the user is

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-16 Thread Chris Withers
Roché Compaan wrote: This is what I am arguing but I haven't had anybody agree/disagree with me yet. It is also a lot simpler to fix: return self.aq_parent.restrictedTraverse(self.getPath(), None) --- return self.aq_parent.unrestrictedTraverse(self.getPath(), None) I don't really mind

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-12 Thread Dieter Maurer
Max M wrote at 2005-3-11 19:10 +0100: ... A single method might be public, but the rest of the object is hidden. What to do then? Just ignore the public method and use the objects overall visibility? The object has a ObjectPermission that controls handling references (!) to the object (itself)

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-11 Thread Chris Withers
Florent Guillaume wrote: In the current getObject problem that concerns us, we want to do better that restrictedTraverse, Why? As far as any problems I had go, it was purely the returning None when the user can see the object that was the bug. Provided getObject raises unauthorised when a user

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-11 Thread Roché Compaan
On Fri, 2005-03-11 at 15:47 +, Chris Withers wrote: Florent Guillaume wrote: In the current getObject problem that concerns us, we want to do better that restrictedTraverse, Why? As far as any problems I had go, it was purely the returning None when the user can see the object that

[Zope-dev] Re: ZCatalog getObject broken

2005-03-11 Thread Max M
Roché Compaan wrote: The rest of the discussion basically boils down to figure out if the user is allowed to access C or not. Hasn't it been raised allready that there is no way of knowing that? A single method might be public, but the rest of the object is hidden. What to do then? Just ignore the

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-11 Thread Roché Compaan
On Fri, 2005-03-11 at 19:10 +0100, Max M wrote: Roché Compaan wrote: The rest of the discussion basically boils down to figure out if the user is allowed to access C or not. Hasn't it been raised allready that there is no way of knowing that? A single method might be public, but the

[Zope-dev] Re: ZCatalog getObject broken

2005-03-10 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Florent Guillaume wrote: | Dieter Maurer [EMAIL PROTECTED] wrote: | |Roché Compaan wrote at 2005-2-25 17:22 +0200: | | Last year in March the following checkin was made that changed | ZCatalog's getObject to use restrictedTraverse instead of |

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-10 Thread Chris McDonough
I implemented a publisherTraverse function like this FWIW: def publisherTraverse(context, path): # this is a hack to get around the fact that restrictedTraverse, # unlike publisher traversal, does checks at every step of the # path. We don't want to limit access in this way (e.g.

[Zope-dev] Re: ZCatalog getObject broken

2005-03-10 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Chris McDonough wrote: | I implemented a publisherTraverse function like this FWIW: | | def publisherTraverse(context, path): | # this is a hack to get around the fact that restrictedTraverse, | # unlike publisher traversal, does checks at

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-10 Thread Roché Compaan
On Thu, 2005-03-10 at 12:13 -0500, Tres Seaver wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Chris McDonough wrote: | I implemented a publisherTraverse function like this FWIW: | | def publisherTraverse(context, path): | # this is a hack to get around the fact that

[Zope-dev] Re: ZCatalog getObject broken

2005-03-03 Thread Max M
Roché Compaan wrote: I'm unsure about the security check in the patch below - I copied the way restrictedTraverse does it. I read through validate in the default security policy but it is one of those methods where all the security implications doesn't fit in your head all at once. ---

Re: [Zope-dev] Re: ZCatalog getObject broken

2005-03-03 Thread Roché Compaan
On Thu, 2005-03-03 at 09:27 +0100, Max M wrote: Roché Compaan wrote: I'm unsure about the security check in the patch below - I copied the way restrictedTraverse does it. I read through validate in the default security policy but it is one of those methods where all the security

[Zope-dev] Re: ZCatalog getObject broken

2005-02-25 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Andreas Jung wrote: | | | --On Freitag, 25. Februar 2005 20:21 Uhr +0100 Dieter Maurer | [EMAIL PROTECTED] wrote: | | Roché Compaan wrote at 2005-2-25 17:22 +0200: | | Last year in March the following checkin was made that changed | ZCatalog's