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 )