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: multiple CVEs #69

Merged
merged 3 commits into from
Jul 27, 2024
Merged

fix: multiple CVEs #69

merged 3 commits into from
Jul 27, 2024

Conversation

ExampleWasTaken
Copy link
Contributor

@ExampleWasTaken ExampleWasTaken commented Jun 25, 2024 •
edited
Loading

Description

This PR fixes:

Test Results

  • Build: ✅
  • Slash commands: ✅
  • Context commands: ✅
  • Action Rows: ✅

@ExampleWasTaken
Copy link
Contributor Author

The relevance of the vulnerabilities reported for the ip package has since been questioned and the repo has been archived by its owner over these reports. Nevertheless, I think it's always good to keep `npm audit happy if there are viable solutions.

@pdellaert pdellaert self-assigned this Jul 11, 2024
@pdellaert
Copy link
Collaborator

Needs testing, but likely will be fine, will get to it soon (likely weekend)

@benw202 benw202 merged commit caa3e23 into flybywiresim:staging Jul 27, 2024
3 checks passed
@ExampleWasTaken ExampleWasTaken deleted the fix/security-1 branch July 28, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benw202 benw202 benw202 approved these changes

Assignees

@pdellaert pdellaert

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ExampleWasTaken @pdellaert @benw202