-
Notifications
You must be signed in to change notification settings - Fork 157
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
[Snyk] Security upgrade react-cookie from 4.1.1 to 8.0.1 #5681
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5681 +/- ##
==========================================
- Coverage 25.66% 25.64% -0.03%
==========================================
Files 474 481 +7
Lines 50905 51244 +339
==========================================
+ Hits 13066 13141 +75
- Misses 36848 37104 +256
- Partials 991 999 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@hongky-1994 would you please check this PR? Not in a hurry. even test/web fails 😓 |
@t-kikuc san, References: |
d0b4278
to
08336a5
Compare
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-COOKIE-8163060 Signed-off-by: t-kikuc <[email protected]>
- Wrap CookiesProvider to root component - Wrap CookiesProvider to error test that use cookies Signed-off-by: kypham <[email protected]> Signed-off-by: t-kikuc <[email protected]>
08336a5
to
2a4062e
Compare
@hongky-1994 Thank you so much for investigating and fixing it!! |
Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):
web/package.json
web/yarn.lock
Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the
.yarn/cache/
directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to runyarn
to update the contents of the./yarn/cache
directory.If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:
SNYK-JS-COOKIE-8163060
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)