[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [jetty-dev] ProxyServlet customizeExchange : add parameter response ?
|
master pls
--
jesse mcconnell
jesse.mcconnell@xxxxxxxxx
On Thu, Mar 1, 2012 at 06:43, Thomas SEGISMONT <tsegismont@xxxxxxxxx> wrote:
> Hi,
>
> Thanks for the quick answer and sorry for waking you up ;)
>
> I'll push a changeset to gerrit. Which branch should I base the change on ?
>
> Le 01/03/2012 12:48, Jesse McConnell a écrit :
>
>> we will certainly take a look, need a coffee before I consider it
>> right or not :)
>>
>> feel free to open a bugzilla and attached the patch or push the patch
>> into gerrit for review
>>
>> https://git.eclipse.org/r/#/q/project:jetty/org.eclipse.jetty.project,n,z
>>
>> cheers,
>> jesse
>>
>> --
>> jesse mcconnell
>> jesse.mcconnell@xxxxxxxxx
>>
>>
>>
>> On Thu, Mar 1, 2012 at 05:02, Thomas SEGISMONT<tsegismont@xxxxxxxxx>
>> wrote:
>>>
>>> Hi,
>>>
>>> I'm trying to build a http transparent proxy, similar to what you get
>>> with
>>> Apache mod_proxy, ProxyPass and ProxyPassReverse. So I need to intercept
>>> response headers sent by the back-end server ( Location, Content-Location
>>> and URI).
>>>
>>> I have a servlet which extends ProxyServlet.Transparent and overrides
>>> method
>>> customizeExchange :
>>>
>>> @Override
>>> protected void customizeExchange(HttpExchange exchange,
>>> HttpServletRequest
>>> request)
>>> {
>>> exchange.setEventListener(new
>>> ProxyEventListener(exchange.getEventListener(), request));
>>> }
>>>
>>> Here I miss a reference to the HttpServletResponse instance. I'd like to
>>> have :
>>>
>>> @Override
>>> protected void customizeExchange(HttpExchange exchange,
>>> HttpServletRequest
>>> request, HttpServletResponse response)
>>> {
>>> exchange.setEventListener(new
>>> ProxyEventListener(exchange.getEventListener(), request, response));
>>> }
>>>
>>> Would you accept a patch like this ? Or do you think there's another way
>>> to
>>> do the job ?
>>>
>>> Thanks
>>> _______________________________________________
>>> jetty-dev mailing list
>>> jetty-dev@xxxxxxxxxxx
>>> https://dev.eclipse.org/mailman/listinfo/jetty-dev
>>
>> _______________________________________________
>> jetty-dev mailing list
>> jetty-dev@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/jetty-dev
>
>
> _______________________________________________
> jetty-dev mailing list
> jetty-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/jetty-dev
