Skip to content

Conversation

@marcelotto
Copy link
Contributor

This PR fixes a bug in URI.merge/2 that was revealed via the W3C test suite of the JSON-LD 1.1 algorithms. The merging with a base ending in dot segments without a trailing slash was not handled correctly.

Example:

URI.merge("http://a/bb/ccc/.", "g") |> to_string()
# Current: "http://a/bb/ccc/+/g"  (incorrect)
# Expected: "http://a/bb/ccc/g"   (correct)

@josevalim josevalim merged commit b9f012f into elixir-lang:main Mar 18, 2025
10 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants