Dieter Maurer wrote:
Tres Seaver wrote at 2004-1-23 17:04 -0500:
The script being called is invoked during a workflow transition, which calls the script, passing a StateChangeInfo instance; the new code tries to validate access to the SCI and pukes, badly, because the script has proxy roles, but the 'container' isn't wrapped. I *think* that we are seeing this issue because the new security code is tougher about checking access within the script.


But why is a "StateChangeInfo" object passed as "container" to the code
above?

Just for clarity: the 'container' in play here is not the object bound as 'container' to the script; it is the argument passed to validate, which is trying to check that the script is allowed.


Up to now, I was convinced that "container" is usually (for acquisition
wrappers) the equivalent to "*.aq_inner.aq_parent" (which I would prefer
to have an "aq_container" for).

Hopefully, we get soon (for Zope 2.9, at least) a clear specification
when "validate" is called with which arguments. It will help
everyone.


I tried to work around the issue by adding a 'getPhysicalRoot' method to SCI, merely delegating to the underlying object; however, the script in my app then blows up on access to 'object' (which I may be able to fix as well).


I think, the security context should provide access to the root.
"container" and "object" can be any type of objects neither of
which may have access to the root.

I am working on a fix for both parts of this; the code in question is that part of validate which is working to ensure that a script with proxy roles is not acquiring an object "above" the current user's acl_users.


Tres.
--
===============================================================
Tres Seaver                                [EMAIL PROTECTED]
Zope Corporation      "Zope Dealers"       http://www.zope.com

_______________________________________________
Zope-Dev maillist - [EMAIL PROTECTED]
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 )

Reply via email to