Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

Update testify to version that resolves CVE-2022-28948 #222

Merged
merged 1 commit into from
Jun 13, 2022
Merged

Conversation

nathancoleman
Copy link
Member

@nathancoleman nathancoleman commented Jun 13, 2022

Changes proposed in this PR:

Our current version of Testify is pulling in a version of gopkg.in/yaml.v3 that contains CVE-2022-28948. The latest version of testify resolves this by updating its yaml.v3 dependency w/ stretchr/testify#1192

$ go mod why gopkg.in/yaml.v3
# gopkg.in/yaml.v3
github.com/hashicorp/consul-api-gateway/internal/adapters/consul
github.com/hashicorp/consul-api-gateway/internal/adapters/consul.test
github.com/stretchr/testify/assert
gopkg.in/yaml.v3

How I've tested this PR:

How I expect reviewers to test this PR:

Checklist:

  • Tests added
  • CHANGELOG entry added

    Run make changelog-entry for guidance in authoring a changelog entry, and
    commit the resulting file, which should have a name matching your PR number.
    Entries should use imperative present tense (e.g. Add support for...)

Our current version of Testify is pulling in a version of gopkg.in/yaml.v3 that contains CVE-2022-28948. The latest version of testify resolves this by updating its yaml.v3 dependency
@nathancoleman nathancoleman added the pr/no-changelog Skip the CI check that requires a changelog entry label Jun 13, 2022
@nathancoleman nathancoleman merged commit a0e5782 into main Jun 13, 2022
@nathancoleman nathancoleman deleted the yaml-cve branch June 13, 2022 17:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr/no-changelog Skip the CI check that requires a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants