Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

trurl works on urls ... what about URIs ? #372

Open
JimFuller-RedHat opened this issue Sep 19, 2024 · 4 comments
Open

trurl works on urls ... what about URIs ? #372

JimFuller-RedHat opened this issue Sep 19, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@JimFuller-RedHat
Copy link

not trolling - just curious about opinions on this.

@bagder
Copy link
Member

bagder commented Sep 19, 2024

How do you define the difference?

@JimFuller-RedHat
Copy link
Author

to take a more concrete use case, consider pURL (https://github.com/package-url/purl-spec)

> trurl pkg:bitbucket/birkenfeld/pygments-main@244fd47e07d1014f0aed9c

gives us

trurl note: Port number was not a decimal number between 0 and 65535 [pkg:bitbucket/birkenfeld/pygments-main@244fd47e07d1014f0aed9c]

@bagder
Copy link
Member

bagder commented Sep 19, 2024

URL/URI schemes that do not use :// are not supported by trurl because the libcurl URL API does not support those.

I don't think it is completely unthinkable that non-hierarchical schemes could be supported as well in a future, but doing that also opens up a can of worms do deal with.

@bagder bagder added the enhancement New feature or request label Sep 19, 2024
@jacobmealey
Copy link
Contributor

I don't think it is completely unthinkable that non-hierarchical schemes could be supported as well in a future, but doing that also opens up a can of worms do deal with.

This reminds me of a previous conversation had on a pr (#278) where pulling out the url API into its own library was suggested. That PR was focused around security and sandboxing, but I think this is a another potential use case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants