Skip to content

Commit

Permalink
Bump dependency versions to fix security issues
Browse files Browse the repository at this point in the history
  • Loading branch information
Leonidas-from-XIV committed Apr 9, 2023
1 parent 3a8d46e commit bd0f780
Showing 1 changed file with 157 additions and 189 deletions.

4 comments on commit bd0f780

@markstos
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This upgrades every module it seems. Which ones had the security issues?

@Leonidas-from-XIV
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@markstos I don't remember, I used npm audit to fix them, without spending too much time looking at it in detail.

@markstos
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found it, I think:

#664

@Leonidas-from-XIV
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, this was a security issue with the code itself, not the dependencies and merged a bit before that.

Please sign in to comment.