Skip to content

Conversation

@LakshmiNarayananDesikan
Copy link
Contributor

@LakshmiNarayananDesikan LakshmiNarayananDesikan commented Sep 17, 2025

Description

Backport of #22581

https://github.com/mitchellh/mapstructure is now a public archive, this PR aims to migrate to a fork of it https://github.com/go-viper/mapstructure (v2) which is maintained and gets frequent security patches.

Ref: PSA-2107

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

* deps: migrate mapstructure to go-viper

* go mod tidy

* add changelog

* fix test

* go mod tidy

* fix test

* fix test

* go mod tidy
@LakshmiNarayananDesikan LakshmiNarayananDesikan requested a review from a team as a code owner September 17, 2025 15:10
@github-actions github-actions bot added theme/api Relating to the HTTP API interface theme/cli Flags and documentation for the CLI interface theme/config Relating to Consul Agent configuration, including reloading theme/connect Anything related to Consul Connect, Service Mesh, Side Car Proxies pr/dependencies PR specifically updates dependencies of project theme/envoy/xds Related to Envoy support theme/contributing Additions and enhancements to community contributing materials theme/internals Serf, Raft, SWIM, Lifeguard, Anti-Entropy, locking topics and removed pr/no-backport labels Sep 17, 2025
@LakshmiNarayananDesikan LakshmiNarayananDesikan added the pr/no-changelog PR does not need a corresponding .changelog entry label Sep 17, 2025
@LakshmiNarayananDesikan LakshmiNarayananDesikan changed the title deps: migrate mapstructure to go-viper (#22581) Backport: deps: migrate mapstructure to go-viper (#22581) Sep 17, 2025
@LakshmiNarayananDesikan LakshmiNarayananDesikan added pr/no-backport pr/dependencies PR specifically updates dependencies of project and removed pr/dependencies PR specifically updates dependencies of project labels Sep 17, 2025
@LakshmiNarayananDesikan LakshmiNarayananDesikan merged commit 15aa895 into release/1.21.x Sep 17, 2025
282 of 337 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/dependencies PR specifically updates dependencies of project pr/no-backport pr/no-changelog PR does not need a corresponding .changelog entry theme/api Relating to the HTTP API interface theme/cli Flags and documentation for the CLI interface theme/config Relating to Consul Agent configuration, including reloading theme/connect Anything related to Consul Connect, Service Mesh, Side Car Proxies theme/contributing Additions and enhancements to community contributing materials theme/envoy/xds Related to Envoy support theme/internals Serf, Raft, SWIM, Lifeguard, Anti-Entropy, locking topics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants