GSIP 39: pluggable url manglers

Btw, the patch is against the current trunk. It should backport more or less nicely, besides the WMS module and the rest modules that have seen some changes.

One specific thing with this patch is that the url encoding of parameters is done by URLEncoder.
That turns the commas into %2C. Now, I've tried with some online url encoders and it would seem the comma and column have to be encoded:
http://www.blooberry.com/indexdot/html/topics/urlencoding.htm
http://www.permadi.com/tutorial/urlEncoding/
http://en.wikipedia.org/wiki/Percent-encoding

and also here is an online encoder:
http://www.albionresearch.com/misc/urlencode.php

However that changes the resutling url a bit (the browsers still seem happy to use them)... if that is a problem we can use a replacement such as http://www.w3.org/International/O-URL-code.html, which I did not use because of the advertisement clause in the license (but that class is short and can be rewritten quickly)

Patch looks great. The only things I would mention are minor:

1) Just remove dead code instead of commenting out
2) I still think buildURL() is a bit of a deceptive method name on ResponseUtils, since it gives no mention of the mangling that is occurring. Just looking at the name of the method without any javadoc context i would just think the method should give me a url only from the components passed in.

Both are not strong points by any means, so as you please. Nice work.

The method actually does both, it mangles and builds the URL at the same time. If there are no registered (or active manglers) it's just pure building, otherwise it's both.
Calling it just mangleURL would be strange as well, as what you pass in is not really a URL, but the pieces of it.
So... not sure... buildAndMangleURL does not sound very good.

As for the dead code, yup, I'll remove it (I thought I removed it all but some escaped my attention)

I actually kind of like buildAndMangleURL() as a name but i fine with just leaving it buildURL(), very minor. So I would say commit at your leisure.

Fixed on trunk, opened another jira to have this backported on 1.7.x