Skip to content

Conversation

Gijsreyn
Copy link
Contributor

PR Summary

This pull request adds the uri() function with the url crate. It also contains the reference documentation.

There's one rough edge case that didn't work, which was on IPv6. See image below.

image

If this needs to be addressed, I need to debug it further. The script used to compare .NET vs dsc is attached.

test_uri_comparison.txt

PR Context

Partially addresses #57.

@SteveL-MSFT
Copy link
Member

@Gijsreyn so in your examples, those ipv6 addresses are the same, just one is long form and one is short form. In this case, you could use [IPAddress]::parse().tostring() to get the short form and compare that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants