Am 29.06.2012 18:49, schrieb ArtemGr: > Tommi Mäkitalo <tommi@...> writes: >> Thank you for the patch. I changed it a little. >> >> Not that pathinfo do not need to have a '/' as a first character. You >> can set the path info in the configuration. Add <pathinfo>$1</pathinfo> >> into the mapping above and it gets the path without the trailing '/'. >> The proxy module now handles both cases (double slash and no slash) >> correctly. >> >> And the custom port works here as expected. >> >> Tommi > Thanks, Tommi! > Here is another patch: https://gist.github.com/2987769#file_proxy_diff_02.cpp > POST parameters should stay in the request body. Passing qparam.getUrl to the > url interferes with the request encoding (for non-ASCII requests) and with the > maximum request length. > True. I just looked at the patch and rechecked. I'm not sure, if "request.getPathInfo()" is the right method. There is a "request.getUrl()", which returns the original url from the request instead of the path info from the configuration.
Maybe I should really rethink this proxy component. But first I am offline for a week. Tommi ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Tntnet-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/tntnet-general
