leejw51 / protobuf-net

Automatically exported from code.google.com/p/protobuf-net
0 stars 0 forks source link

Serialization of some encoded URI's fail #83

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
URI's is currently serialized by calling `ToString()`, where it should use 
`AbsoluteUri` (or rather `OriginalString` to be compatible with relative 
URI's as well).

`ToString()` isn't guaranteed to return a string that will recreate an equal 
URI instance. For further information about this behaviour take a look at the 
following links, or the attached test case:

* http://blog.nerdbank.net/2008/04/uriabsoluteuri-and-uritostring-are-
not.html
* http://blogs.msdn.com/yangxind/archive/2006/11/09/don-t-use-net-system-uri-
unescapedatastring-in-url-decoding.aspx

Original issue reported on code.google.com by troet...@gmail.com on 16 Nov 2009 at 2:04

Attachments:

GoogleCodeExporter commented 9 years ago
As they say... oops! Cheers for the patch. Will apply this for the next release.

Original comment by marc.gravell on 16 Nov 2009 at 8:51

GoogleCodeExporter commented 9 years ago
Fix implemented; just packaging

Original comment by marc.gravell on 25 Nov 2009 at 10:22

GoogleCodeExporter commented 9 years ago
r278

Original comment by marc.gravell on 25 Nov 2009 at 10:49