Commit 029d91d4 authored by Jim Fulton's avatar Jim Fulton

Added logic to close connections created when doing a retry after a

conflict error.  Without this, conflict errors lead to leaked connections and, eventually, hung sites.
parent 12bd887d
...@@ -84,8 +84,8 @@ ...@@ -84,8 +84,8 @@
############################################################################## ##############################################################################
__doc__="""Python Object Publisher -- Publish Python objects on web servers __doc__="""Python Object Publisher -- Publish Python objects on web servers
$Id: Publish.py,v 1.139 1999/08/18 00:24:46 amos Exp $""" $Id: Publish.py,v 1.140 1999/08/27 23:36:33 jim Exp $"""
__version__='$Revision: 1.139 $'[11:-2] __version__='$Revision: 1.140 $'[11:-2]
import sys, os import sys, os
from string import lower, atoi, rfind, strip from string import lower, atoi, rfind, strip
...@@ -186,7 +186,12 @@ def publish(request, module_name, after_list, debug=0, ...@@ -186,7 +186,12 @@ def publish(request, module_name, after_list, debug=0,
sys.exc_info()[1], sys.exc_info()[1],
sys.exc_info()[2], sys.exc_info()[2],
) )
return publish(request.retry(), module_name, after_list, debug) newrequest=request.retry()
try:
return publish(newrequest, module_name, after_list, debug)
finally:
newrequest.close()
else: raise else: raise
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment