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

Correctly parse file URIs #421

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

Conversation

Richard-W
Copy link

This changes the URI parser to allow URIs of the forms

  • scheme:/absolute/path
  • scheme:///absolute/path

It does impact HTTP URI parsing in that HTTP URIs without an authority
part are now allowed.

Fixes issue #323

Richard-W added 2 commits May 16, 2020 13:57
This changes the URI parser to allow URIs of the forms
* scheme:/absolute/path
* scheme:///absolute/path

It does impact HTTP URI parsing in that HTTP URIs without an authority
part are now allowed.

Fixes issue hyperium#323
@oblique
Copy link

oblique commented Apr 14, 2021

Any updates on this?

@0ndorio
Copy link

0ndorio commented Oct 8, 2021

@seanmonstar Is this merge request still blocked by something or is there anything I could do to help this to be merged?

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.

3 participants