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

Should fragment been encoded? #18

Open

qianL93 opened this issue Oct 12, 2020 · 0 comments

qianL93 commented Oct 12, 2020

query-string-for-all/index.js

Line 374 in d8aa8fd

hash = `#${encode(input.fragmentIdentifier, options)}`;

According to mdn, maybe it is unnecessary to encode fragment here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment