From e7b6e2a402922724ffd668161ed1b65533b5c034 Mon Sep 17 00:00:00 2001 From: David Kelly Date: Wed, 1 Feb 2012 14:08:29 -0800 Subject: Added absoluteURI support If requests go through a proxy, they will have the original uri in the request, i.e. : GET http://proxy.server.uri/some/query/string HTTP 1.1 ... That was problematic -- cowboy_http_protocol:request didn't know what to to with the result of decode_packet applied to this, which would be something like: ``` erlang {http_request,'GET',{absoluteURI,http,<<"proxy.server.uri">>, undefined,<<"/some/query/string">>},{1,1}} ``` So, I just ignore the host, grab the path and pass into ``` erlang cowboy_http_protocol:request({http_request, Method, {abs_path, Path}, Version}, State) ``` Seems to do the trick without much effort. --- test/http_SUITE.erl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'test') diff --git a/test/http_SUITE.erl b/test/http_SUITE.erl index 862a110..9191f26 100644 --- a/test/http_SUITE.erl +++ b/test/http_SUITE.erl @@ -378,7 +378,8 @@ raw(Config) -> {"GET / HTTP/1.1\r\nHost: localhost", 408}, {"GET / HTTP/1.1\r\nHost: localhost\r\n", 408}, {"GET / HTTP/1.1\r\nHost: localhost\r\n\r", 408}, - {"GET http://localhost/ HTTP/1.1\r\n\r\n", 501}, + {"GET http://proxy/ HTTP/1.1\r\n\r\n", 400}, + {"GET http://proxy/ HTTP/1.1\r\nHost: localhost\r\n\r\n", 200}, {"GET / HTTP/1.2\r\nHost: localhost\r\n\r\n", 505}, {"GET /init_shutdown HTTP/1.1\r\nHost: localhost\r\n\r\n", 666}, {"GET /long_polling HTTP/1.1\r\nHost: localhost\r\n\r\n", 102}, -- cgit v1.2.3