We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
bd0f780
There was a problem 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?
Sorry, something went wrong.
@markstos I don't remember, I used npm audit to fix them, without spending too much time looking at it in detail.
npm audit
#664
No, this was a security issue with the code itself, not the dependencies and merged a bit before that.
bd0f780
There was a problem hiding this comment.
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?
bd0f780
There was a problem hiding this comment.
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.bd0f780
There was a problem hiding this comment.
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
bd0f780
There was a problem hiding this comment.
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.