Re: ZPublisher events before streaming a response
Hanno Schlichting <[email protected]>
| Newsgroups | gmane.comp.web.zope.devel |
|---|---|
| Message-ID | <[email protected]> |
On Sun, Mar 21, 2010 at 10:07 AM, Martin Aspeli <[email protected]> wrote: > We'd therefore like to add a new event in the HTTPResponse class (in > ZServer, though I think it makes sense to add to the ZPublisher base > class version as well). It'd hook in something like this: > > if not self._wrote: > > # new event code > site = getSite() > request = getattr(site, 'REQUEST', None) > notify(PubBeforeStreaming(request)) > > # continue as before... > > (I couldn't find a better way to get hold of the request from a method > in the response, without adding a dependency on five.globalrequest, > which I assume is not desirable). > > Any objections? We need this in Zope 2.12, though I'll obviously merge > to trunk, too. Why would you need to have the event on the request, if all you want is to set headers? Why not make it an event with the response as the argument instead? Hanno _______________________________________________ Zope-Dev maillist - [email protected] https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )