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

feat: move to built-in URL class #998

Closed
wants to merge 3 commits into from
Closed

Commits on Jan 27, 2024

  1. feat: move to built-in URL class

    We currently use `url-parse` to deal with URL construction. Given our
    node engine constraint of `>=14`, we know `URL` is available in all
    supported node versions (introduced in `10.x` if i remember correctly).
    
    So we should be able to drop this dependency and use the built-in parser
    instead.
    
    Note that one test here changed hash because url-parse incorrectly did
    not encode the single quotes in the test URL. With the standard `URL`
    parsing, it is now encoded.
    43081j committed Jan 27, 2024
    Configuration menu
    Copy the full SHA
    c925cb8 View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2024

  1. Configuration menu
    Copy the full SHA
    49fc889 View commit details
    Browse the repository at this point in the history

Commits on Apr 5, 2024

  1. Configuration menu
    Copy the full SHA
    25b5a26 View commit details
    Browse the repository at this point in the history