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: support ?= for set, to only do it if not already set #298

Closed
wants to merge 2 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented May 13, 2024

Idea-by: @emanuele6 in #296

Due to missing support in the libcurl URL API, this is not really working for the scheme component right now, as that will always be considered set once a URL has been parsed. The scheme guessing the parser does sets it internally and there is currently no way provided by the API to figure that out.

@bagder bagder changed the title trurl: support ?= for set, to only do it conditionally if not already… trurl: support ?= for set, to only do it if not already set May 13, 2024
Idea-by: @emanuele6 in #296

Due to missing support in the libcurl URL API, this is not really
working for the scheme component right now, as that will always be
considered set once a URL has been parsed. The scheme guessing the
parser does sets it internally and there is currently no way provided by
the API to figure that out.
@bagder bagder closed this in 13d7449 May 13, 2024
@bagder bagder deleted the bagder/conditional-set branch May 13, 2024 18:17
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.

1 participant