End request handling lost when 'abort' raised an exception

[Dieter Maurer]

"ZPublisher.Publish" contains code in two places of the form

                if transactions_manager:
                    transactions_manager.abort()
                endInteraction()

As "abort" can raise an exception, the "endInteraction" is not called in such cases (I am unsure whether this is fatal).

I suggest to use instead code of the form:

            # Note: 'abort's can fail. Nevertheless, we want end request handling
            try:
                if transactions_manager:
                    transactions_manager.abort()
            finally:
                endInteraction()

[Hanno Schlichting (hannosch)]

This is fixed in at least Zope 2.12, where the try/finally has been introduced in combination with the extra PubFailure event.