On Fri, 2004-05-14 at 22:49, Michael Dunstan wrote:

> hmm... I suspect there are few holes in this...
> 
> - Need to make sure there is no get_transaction().begin() in  
> zpublisher_exception_hook() of lib/python/Zope/App/startup.py
> 
>    That begin() would mean that the error handling is not in fact part  
> of the original transaction?

Right.

> - An exception within err_hook() (aside from a Retry) will mean that  
> abort() is never called.

Ya, thanks.

Dieter, do you think you can read this patch and give a thumbs up or
down on it?  (If SteveA and/or Toby are reading, please do same?)

On a different subject, the publisher probably shouldn't pass around
traceback objects (e.g. when it calls into err_hook) as Tres believes
that may be a memory leak waiting to happen.  I will first apply the
error-in-main-transaction patch to the 2.7 branch and worry about that
later however.

- C



_______________________________________________
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