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

Bug Fix Needed - [email protected] requires [email protected] via a transitive dependency on [email protected] #314

Closed
jeremiahjordanisaacson opened this issue Dec 16, 2024 · 0 comments
Assignees
Labels
bug Something isn't working security This update patches security vulnerabilities.

Comments

@jeremiahjordanisaacson
Copy link
Member

Dependabot alerts #99
cookie accepts cookie name, path, and domain with out of bounds characters #99
Open Opened 2 months ago on cookie (npm) · package-lock.json
Dependabot cannot update cookie to a non-vulnerable version
The latest possible version that can be installed is 0.6.0 because of the following conflicting dependencies:

[email protected] requires [email protected] via a transitive dependency on [email protected]
No patched version available for cookie
The earliest fixed version is 0.7.0.

Package
Affected versions
Patched version
cookie
(npm)
< 0.7.0
0.7.0
Impact
The cookie name could be used to set other fields of the cookie, resulting in an unexpected cookie value. For example, serialize("userName=<script>alert('XSS3')</script>; Max-Age=2592000; a", value) would result in "userName=<script>alert('XSS3')</script>; Max-Age=2592000; a=test", setting userName cookie to <script> and ignoring value.

A similar escape can be used for path and domain, which could be abused to alter other fields of the cookie.

Patches
Upgrade to 0.7.0, which updates the validation for name, path, and domain.

Workarounds
Avoid passing untrusted or arbitrary values for these fields, ensure they are set by the application instead of user input.

References
jshttp/cookie#167

@jeremiahjordanisaacson jeremiahjordanisaacson added bug Something isn't working security This update patches security vulnerabilities. labels Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working security This update patches security vulnerabilities.
Projects
None yet
Development

No branches or pull requests

1 participant