Mercurial > public > mercurial-scm > hg-stable
diff tests/hgweberror.py @ 36882:97f44b0720e2
hgweb: port archive command to modern response API
Well, I tried to go with PEP 3333's recommendations and only allow
our WSGI application to emit data via a response generator.
Unfortunately, the "archive" command calls into the zipfile and
tarfile modules and these operator on file objects and must send
their data to an object with write(). There's no easy way turn
these write() calls into a generator.
So, we teach our response type how to expose a file object like
object that can be used to write() output. We try to keep the API
consistent with how things work currently: callers must call a
setbody*(), then sendresponse() to trigger sending of headers,
and only then can they get a handle on the object to perform
writing.
This required overloading the return value of @webcommand functions
even more. Fortunately, we're almost completely ported off the
legacy API. So we should be able to simplify matters in the near
future.
A test relying on this functionality has also been updated to use
the new API.
Differential Revision: https://phab.mercurial-scm.org/D2792
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Sat, 10 Mar 2018 20:16:20 -0800 |
parents | 3d60a22e27f5 |
children | 4daa22071d5d |
line wrap: on
line diff
--- a/tests/hgweberror.py Sat Mar 10 16:17:51 2018 -0800 +++ b/tests/hgweberror.py Sat Mar 10 20:16:20 2018 -0800 @@ -10,9 +10,12 @@ '''Dummy web command that raises an uncaught Exception.''' # Simulate an error after partial response. - if 'partialresponse' in req.req.qsparams: - req.respond(200, 'text/plain') - req.write('partial content\n') + if 'partialresponse' in web.req.qsparams: + web.res.status = b'200 Script output follows' + web.res.headers[b'Content-Type'] = b'text/plain' + web.res.setbodywillwrite() + list(web.res.sendresponse()) + web.res.getbodyfile().write(b'partial content\n') raise AttributeError('I am an uncaught error!')