Skip to content

Update codeclysm/extract dependency #601

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
Merged
umbynos merged 2 commits into from
Jan 20, 2021
Merged

Update codeclysm/extract dependency #601

umbynos merged 2 commits into from
Jan 20, 2021

Conversation

@umbynos
Copy link
Contributor

@umbynos umbynos commented Jan 19, 2021

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • What kind of change does this PR introduce?

update deps

  • What is the current behavior?
  • What is the new behavior?

update codeclysm/extract dependency to v3 to fix zipslip vulnerability (codeclysm/extract#14)

  • Does this PR introduce a breaking change?
  • Other information:

@umbynos umbynos self-assigned this Jan 19, 2021
@umbynos umbynos added dependencies topic: code Related to content of the project itself labels Jan 19, 2021
@umbynos umbynos requested a review from zmoog January 19, 2021 16:19
zmoog
zmoog approved these changes Jan 20, 2021
View reviewed changes
Copy link
Contributor

@zmoog zmoog left a comment

Choose a reason for hiding this comment

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

LGTM

@umbynos umbynos merged commit aac77f7 into devel Jan 20, 2021
@umbynos umbynos deleted the umbynos/update_extract_deps branch January 20, 2021 09:52
umbynos added a commit that referenced this pull request Jan 29, 2021
@umbynos
Update codeclysm/extract dependency (#601)
8c8f2d3 
* update github.com/codeclysm/extract to v3 (fix zipslip vulnerability)

* run go mod tidy
@rsora rsora added the topic: infrastructure Related to project infrastructure label Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@zmoog zmoog zmoog approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@umbynos umbynos

Labels

topic: code Related to content of the project itself topic: infrastructure Related to project infrastructure

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@umbynos @zmoog @rsora