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

fix: proper URL escaping and wrapping (url()) #627

Merged
merged 8 commits into from Jan 4, 2018
Merged

fix: proper URL escaping and wrapping (url()) #627

merged 8 commits into from Jan 4, 2018

Commits on Oct 31, 2017

  1. Escape invalid urls

    @kgram
    kgram committed Oct 31, 2017
    Copy the full SHA
    d14e8b8 View commit details
    Browse the repository at this point in the history

  2. Correct existing tests to expect double-quotes

    @kgram
    kgram committed Oct 31, 2017
    Copy the full SHA
    2db8469 View commit details
    Browse the repository at this point in the history

  3. Add url-escape specific tests

    @kgram
    kgram committed Oct 31, 2017
    Copy the full SHA
    74431ba View commit details
    Browse the repository at this point in the history

  4. Switch to regex to detect quote-wrapped urls

    @kgram
    kgram committed Oct 31, 2017
    Copy the full SHA
    fdac0b0 View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2017

  1. Rename and rearrange url-escape variables

    @kgram
    kgram committed Nov 1, 2017
    Copy the full SHA
    1ac7661 View commit details
    Browse the repository at this point in the history

  2. Rename lib/escape-url.js -> lib/url/escape.js

    @kgram
    kgram committed Nov 1, 2017
    Copy the full SHA
    87caf33 View commit details
    Browse the repository at this point in the history

  3. Add semicolon to url escape-helper output

    @kgram
    kgram committed Nov 1, 2017
    Copy the full SHA
    d34841f View commit details
    Browse the repository at this point in the history

  4. Add test for url-loader output

    @kgram
    kgram committed Nov 1, 2017
    Copy the full SHA
    f144280 View commit details
    Browse the repository at this point in the history