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

urlapi: add CURLU_NO_GUESS_SCHEME #13616

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bagder
Copy link
Member

@bagder bagder commented May 13, 2024

Used for extracting:

  • when used asking for a scheme, it will return CURLUE_NO_SCHEME if the stored information was a guess

  • when used asking for a URL, the URL is returned without a scheme, like when previously given to the URL parser when it was asked to guess

  • as soon as the scheme is set explicitly, it is no longer internally marked as guessed

The idea being:

  1. allow a user to figure out if a URL's scheme was set as a result of guessing

  2. extract the URL without a guessed scheme

  3. this makes it work similar to how we already deal with port numbers

Extend test 1560 to verify.

Used for extracting:

- when used asking for a scheme, it will return CURLUE_NO_SCHEME if the
  stored information was a guess

- when used asking for a URL, the URL is returned without a scheme, like
  when previously given to the URL parser when it was asked to guess

- as soon as the scheme is set explicitly, it is no longer internally
  marked as guessed

The idea being:

1. allow a user to figure out if a URL's scheme was set as a result of
  guessing

2. extract the URL without a guessed scheme

3. this makes it work similar to how we already deal with port numbers

Extend test 1560 to verify.
@icing
Copy link
Contributor

icing commented May 13, 2024

CURLU_CLUELESS.

@bagder bagder added feature-window A merge of this requires an open feature window labels May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-window A merge of this requires an open feature window libcurl API tests URL
Development

Successfully merging this pull request may close these issues.

None yet

2 participants