Skip to content

Commit 84564bf

Browse files
authored
Merge pull request MessagePack-CSharp#1106 from degant/depuri/fix1105
UriFormatter should use Uri.OriginalString when serializing
2 parents 1869199 + 185e123 commit 84564bf

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

src/MessagePack.UnityClient/Assets/Scripts/MessagePack/Formatters/StandardClassLibraryFormatter.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ public void Serialize(ref MessagePackWriter writer, Uri value, MessagePackSerial
300300
}
301301
else
302302
{
303-
writer.Write(value.ToString());
303+
writer.Write(value.OriginalString);
304304
}
305305
}
306306

src/MessagePack.UnityClient/Assets/Scripts/Tests/ShareTests/FormatterTest.cs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -294,11 +294,14 @@ public void DecimalLang()
294294
}
295295
}
296296

297-
[Fact]
298-
public void UriTest_Absolute()
297+
[Theory]
298+
[InlineData("http://google.com/")]
299+
[InlineData("http://google.com:80/")]
300+
[InlineData("https://example.com:443/")]
301+
public void UriTest_Absolute(string url)
299302
{
300-
var absolute = new Uri("http://google.com/");
301-
this.Convert(absolute).ToString().Is("http://google.com/");
303+
var absolute = new Uri(url);
304+
this.Convert(absolute).OriginalString.Is(url);
302305
}
303306

304307
[SkippableFact]

0 commit comments

Comments
 (0)