Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jetty-dev] ProxyServlet customizeExchange : add parameter response ?

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



Back to the top