Hi Rémi,
 
Thanks for the patch, that helped me fix this issue more quickly. I added more 
documentation, reverted the order of "forwardedAddresses" to the one in the 
HTTP header and added a member variable to store the "address" property. Also, 
I've added a "ciua" variable in Resolver to get the upstream address.
 
All changes are checked in SVN trunk.
 
Best regards,
Jérôme Louvel
--
Restlet ~ Founder and Lead developer ~  <http://www.restlet.org/> 
http://www.restlet.org
Noelios Technologies ~ Co-founder ~  <http://www.noelios.com/> 
http://www.noelios.com

  _____  

De : [email protected] [mailto:[email protected]] De la part de Rémi 
Dewitte
Envoyé : vendredi 16 janvier 2009 17:29
À : [email protected]
Objet : Re: HttpRequest suggestion


Patch submitted ! :)

Rémi


On Wed, Jan 14, 2009 at 21:31, Jerome Louvel <[email protected]> wrote:


Hi Rémi,
 
This remark has been made in the past and we have a RFE for this pending:
 
"Add Request#getForwardedAddresses() method"
http://restlet.tigris.org/issues/show_bug.cgi?id=668
 
If you have time to work on a patch that would be welcome! :)
 


Best regards,
Jérôme Louvel
--
Restlet ~ Founder and Lead developer ~  <http://www.restlet.org/> 
http://www.restlet.org
Noelios Technologies ~ Co-founder ~  <http://www.noelios.com/> 
http://www.noelios.com

  _____  

De : [email protected] [mailto:[email protected]] De la part de Rémi 
Dewitte
Envoyé : lundi 12 janvier 2009 17:34
À : [email protected]
Objet : HttpRequest suggestion


Hello all,

When using http forward, I would find more convenient to have the real client 
address using request.getClientInfo().getAddress() instead of the forwarder 
component.

It means changing line 227 of HttpRequest from
result.getAddresses().add(addresses[i].trim());
to 
result.getAddresses().add(0,addresses[i].trim());

What do you think ?

Rémi

------------------------------------------------------
http://restlet.tigris.org/ds/viewMessage.do?dsForumId=4447&dsMessageId=1038963

Reply via email to