[Zope-dev] ZPublisher events before streaming a response

[Hanno Schlichting]

On Sun, Mar 21, 2010 at 10:07 AM, Martin Aspeli
<optilude+lists at gmail.com> 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