Nevow should handle interrupted HTTP responses
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
nevow |
New
|
Undecided
|
Unassigned |
Bug Description
(This is copied from the mailing list. I enclose a patch, which includes a unit test.)
J.P's excellent article http://
explains how to avoid calling Request.finish() on a request after its
connection was lost, because that now raises an exception. The code in
the article is able to avoid calling finish() because it handles the
rendering itself. The call to finish() is right there in _delayedRender.
But I'm using Nevow (because it's awesome) and all that stuff is done
behind the scenes, where I can't get to it.
My application has a form, which uses the Post/Redirect/Get pattern
(http://
double-posting. Because processing is not instant, the browser receives
the HTTP header telling it to redirect before the processing has
finished. When the processing finishes, the Deferred embedded in the web
page fires with a value, Stan finishes flattening the page, and Nevow
calls Request.finish(). By then the browser has closed the connection
and sent in the GET request. So I get an error every single time.
Any page that uses Deferreds and redirects will do as well.
This is a completely self-contained example, to be run with twistd:
from nevow import tags, inevow, loaders, appserver
from nevow.rend import Page
from nevow.url import URL
from twisted.
from twisted.internet import reactor
from twisted.application import service
from twisted.
from twisted.web import http
class Redirector(Page):
count = 0
message = 0
def _increment(self):
self.count += 1
return self.count
def slow_operation(
return deferLater(reactor, 3, self._increment)
def beforeRender(self, ctx):
req = inevow.
if req.method == 'POST':
oldurl = URL.fromContext
newurl = oldurl.
def render_
return self.message
docFactory = loaders.stan(
tags.html[
]
]
)
application = service.
site = appserver.
webservice = TCPServer(8123, site)
webservice.
webservice.