Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update golang.org/x/text to v0.3.3 #3459

Merged
merged 1 commit into from
Aug 10, 2020

Conversation

jmrodri
Copy link
Member

@jmrodri jmrodri commented Jul 18, 2020

Description of the change:
Pull in CVE-2020-14040 fixed by golang.org/x/text v0.3.3.

Motivation for the change:

Checklist

If the pull request includes user-facing changes, extra documentation is required:

@camilamacedo86
Copy link
Contributor

is not it a duplication of #3458?

@jmrodri
Copy link
Member Author

jmrodri commented Jul 20, 2020

is not it a duplication of #3458?

@camilamacedo86 this PR is against the v0.19.x branch.

Copy link
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 20, 2020
@camilamacedo86
Copy link
Contributor

Just rebase it with the 0.19.x branch that it will fix the Markdown check.

@jmrodri jmrodri force-pushed the update-golang-text-019 branch from 8c261ef to f4ed032 Compare July 30, 2020 22:28
@openshift-ci-robot
Copy link

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jul 30, 2020
@estroz estroz closed this Aug 6, 2020
@estroz estroz reopened this Aug 6, 2020
@jmrodri jmrodri merged commit 4dd9248 into operator-framework:v0.19.x Aug 10, 2020
@jmrodri jmrodri deleted the update-golang-text-019 branch August 17, 2020 15:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants