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

add RUSTSEC-2023-0063 to config.toml #4040

Merged
merged 2 commits into from
Sep 21, 2023
Merged

add RUSTSEC-2023-0063 to config.toml #4040

merged 2 commits into from
Sep 21, 2023

Conversation

marcellorigotti
Copy link
Contributor

@marcellorigotti marcellorigotti commented Sep 21, 2023

Pull Request

Closes: PRO-xxx

Checklist

Please conduct a thorough self-review before opening the PR.

  • I am confident that the code works.
  • I have updated documentation where appropriate.

Summary

Added RUSTSEC-2023-0063 to config.toml to ignore the vulnerability (as fare as we know libp2p doesn't use the dependency involved)

@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #4040 (7c50bd6) into main (178df88) will increase coverage by 0%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #4040   +/-   ##
=====================================
  Coverage     72%     72%           
=====================================
  Files        368     368           
  Lines      58484   58489    +5     
  Branches   58484   58489    +5     
=====================================
+ Hits       42327   42331    +4     
  Misses     14060   14060           
- Partials    2097    2098    +1     

see 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@marcellorigotti marcellorigotti marked this pull request as ready for review September 21, 2023 13:06
@dandanlen dandanlen merged commit e94f1f2 into main Sep 21, 2023
44 checks passed
@dandanlen dandanlen deleted the fixRUSTSEC-2023-0063 branch September 21, 2023 13:44
@dandanlen
Copy link
Collaborator

Thanks @marcellorigotti

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants