WMS parameter handling is out-of-spec

[bartvde@…]

E-mail Paul Ramsey:

Finally, since any parameters for a CGI request, either WMS or vanilla
mapserv.cgi, are read and decoded en masse in
 
  cgiutil.c:int loadParams(cgiRequestObj *request)
 
the WMS parameter handling in Mapserver is also out-of-spec. As long as
no one ever defines a LAYER or STYLE name with a "," in it though, no
one will ever notice.  As an added bonus, putting the behavior in-spec
would probably break a largish number of clients that improperly encode
the complete contents of the LAYER, STYLE and BBOX parameters.
 
As a side note, uDig originally did the wrong (but common) thing, of
encoding the full value string. For an in-spec server, this breaks
things. The only server that broke for us was CubeWerx. So in being
out-of-spec Mapserver is in the large majority. Happily, an in-spec
client (that does not encode the "," separator) can interoperate with
an out-of-spec server (that tries to decode the whole parameter), so
now uDig works with both CubeWerx and Mapserver.

[dmorissette]

This ticket is about WMS Server. See also ticket #1296 for related issues in the WMS client.

Both should probably be addressed at the same time (in 5.2?).