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.

Sign up for GitHub

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

Jump to bottom

ne_uri_parse fixes #138

Merged
merged 3 commits into from
Dec 4, 2023
Merged

ne_uri_parse fixes #138

merged 3 commits into from
Dec 4, 2023

Commits on Dec 4, 2023

  1. * src/ne_uri.c (ne_uri_parse): Fix parsing of uri without path but wi… 

    …th query or fragment.

* test/uri-tests.c (parse): Add test cases.

RFC 3986 3.2:
The authority component is preceded by a double slash ("//") and is
terminated by the next slash ("/"), question mark ("?"), or number
sign ("#") character, or by the end of the URI.
    @fasticc
    fasticc committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    cbd4a3a View commit details
    Browse the repository at this point in the history

  2. * src/ne_uri.c (ne_uri_parse): Find port in the forward direction to … 

    …prevent excessive characters from being placed in host.
    @fasticc
    fasticc committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    aed05f6 View commit details
    Browse the repository at this point in the history

  3. * src/ne_uri.c (ne_uri_parse): Check that port contains only digits. 

    * test/uri-test.c (failparse): Add test cases.
    @fasticc
    fasticc committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    b1e8216 View commit details
    Browse the repository at this point in the history