search

jgauffin / Griffin.Framework

Application framework for Business Applications
http://griffinframework.net
168 stars 62 forks source link

fix for UrlDecoder using WebUtility.UrlDecode instead of Uri.UnescapeDa... #10

Closed campersau closed 9 years ago

campersau commented 9 years ago

...taString

perviously if the values contained + (plus) signs they were not properly decoded as spaces

reference: http://blogs.msdn.com/b/yangxind/archive/2006/11/09/don-t-use-net-system-uri-unescapedatastring-in-url-decoding.aspx

NOTE: there is another Uri.UnescapeDataString call in the MultipartSerializer which might be affected too.

campersau commented 9 years ago

Looks like in .NET 4.5 there is an WebUtility.UrlDecode method in the System.Net namespace which also works.

jgauffin commented 9 years ago

What's better with WebUtility.UrlDecode?

campersau commented 9 years ago

Only that it does not require an additional reference (System.Web).

jgauffin commented 9 years ago

thank you for all pull requests. much appreciated.