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

Address CVE-2020-15228 #103

Closed
LCaparelli opened this issue Oct 5, 2020 · 14 comments · Fixed by #99
Closed

Address CVE-2020-15228 #103

LCaparelli opened this issue Oct 5, 2020 · 14 comments · Fixed by #99
Assignees
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code

Comments

@LCaparelli
Copy link

Currently, the action uses the set-env and add-path commands which have recently been deprecated by Github as remediation to $SUBJECT. They have already been replaced with new mechanisms which offer the same functionality. Please refer to:

When running the action users get warnings about the deprecated actions.

@alex
Copy link

alex commented Oct 7, 2020

Limited grepping suggests that the issue here isn't in golangci-lint-action's own source code, but rather in one of the dependencies. Seems not unlikely merely bumping the version will resolve this, but I don't JS packaging well enough to track down which dep it is.

@LCaparelli
Copy link
Author

Limited grepping suggests that the issue here isn't in golangci-lint-action's own source code, but rather in one of the dependencies. Seems not unlikely merely bumping the version will resolve this, but I don't JS packaging well enough to track down which dep it is.

I thought that could be the case as I couldn't find the deprecated commands in this repo either. I'm also unsure how to track this down further.

@tcnghia
Copy link

tcnghia commented Oct 8, 2020

It is action/core: needs to be 1.2.6 or later

https://github.com/golangci/golangci-lint-action/blob/v2/package.json#L29

@sayboras
Copy link
Member

sayboras commented Oct 8, 2020

Thanks for reporting this issue, I will have a chance to take a look weekend only.

PR is much appreciated 💯

@sayboras sayboras added dependencies Pull requests that update a dependency file help wanted Extra attention is needed javascript Pull requests that update Javascript code labels Oct 8, 2020
@tcnghia
Copy link

tcnghia commented Oct 8, 2020

@sayboras it is already fixed at HEAD, so just need a new release I think

@tcnghia
Copy link

tcnghia commented Oct 8, 2020

Was fixed here #96

@sayboras
Copy link
Member

sayboras commented Oct 8, 2020

it is already fixed at HEAD, so just need a new release I think

@tcnghia oh thanks a lot 👍. The testing currently in github action is quite limited, let me do quick check sometime tonight or tomorrow, if it's all good, I will create new release.

@tcnghia
Copy link

tcnghia commented Oct 8, 2020

Thanks a lot @sayboras

@LCaparelli
Copy link
Author

Thank you @tcnghia @sayboras :-)

@tcnghia
Copy link

tcnghia commented Oct 11, 2020

@ernado thanks for merging #99. When do we release a new version that includes this fix? Thanks!

@sayboras
Copy link
Member

@tcnghia yup, I will plan sometimes for releasing new version v2.2.1 today :)

@tcnghia
Copy link

tcnghia commented Oct 11, 2020

Thanks a lot!!

@sayboras
Copy link
Member

v2.2.1 is released https://github.com/golangci/golangci-lint-action/releases/tag/v2.2.1.

I have also tested with one of my private repo, seems good.

@sayboras
Copy link
Member

dependabot is working hard as well golangci/golangci-lint#1447

sternik added a commit to nordcloud/golangci-lint-action that referenced this issue Nov 18, 2020
Fixed "The `set-env` command is disabled"

sugestion from golangci#103 (comment)
@ldez ldez removed the help wanted Extra attention is needed label May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants