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

Update st2client.js to 1.2.2 #203

Merged
merged 2 commits into from
Apr 21, 2020
Merged

Update st2client.js to 1.2.2 #203

merged 2 commits into from
Apr 21, 2020

Conversation

blag
Copy link
Contributor

@blag blag commented Apr 21, 2020

This PR updates st2client.js from 1.1.3 to version 1.2.2, so it pulls in the changes from PR StackStorm/st2client.js#76.

0.10.3
------

* Update st2client.js to 1.2.2 [PR #203] (change)
Copy link
Member

@arm4b arm4b Apr 21, 2020

Choose a reason for hiding this comment

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

Ideally to indicate in the changelog that it's an update to potentially vulnerable dependencies

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I generally try to avoid mentioning this, since people may scan changelogs for security updates, figure out how to attack older ST2 instances, and then start attacking older ST2 installations. The Linux kernel ran into this issue awhile back IIRC.

Copy link
Member

Choose a reason for hiding this comment

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

I disagree.

Users should know whether specific versions of the released software had any potential security issues, fixes, mitigations or not. This could be a driving factor for them to update or not.

I don't think it's a common practice to hide this kind of information behind the vague changelog.


For example here is pip changelog https://pip.pypa.io/en/stable/news/#id302:

Upgrade the bundled copy of requests to 2.6.0, fixing CVE-2015-2296.

This changelog is related to updating pip's dependency requests which had a security issue (https://nvd.nist.gov/vuln/detail/CVE-2015-2296).

@arm4b arm4b added the security label Apr 21, 2020
@blag blag merged commit d2559db into master Apr 21, 2020
@blag blag deleted the update-st2client.js-1.2.2 branch April 21, 2020 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants