Skip to content

Do not use .MustString() in auth interpolation warning - #2412

Merged
shreyas-goenka merged 2 commits into
mainfrom
avoid-must-string
Mar 3, 2025
Merged

Do not use .MustString() in auth interpolation warning#2412
shreyas-goenka merged 2 commits into
mainfrom
avoid-must-string

Conversation

@shreyas-goenka

@shreyas-goenkashreyas-goenka commented Mar 1, 2025

Copy link
Copy Markdown
Contributor

Why

Addresses post merge feedback from #2399 (comment)

Tests

N/A

@shreyas-goenkashreyas-goenka changed the title Do not use MustString in auth interpolation warningDo not use .MustString() in auth interpolation warningMar 1, 2025
@shreyas-goenka
shreyas-goenka marked this pull request as ready for review March 1, 2025 20:15
Comment threadbundle/config/validate/interpolation_in_auth_config.go
@shreyas-goenka
shreyas-goenka added this pull request to the merge queueMar 3, 2025
Merged via the queue into main with commit 25a7e0aMar 3, 2025
@shreyas-goenka
shreyas-goenka deleted the avoid-must-string branch March 3, 2025 12:12
denik pushed a commit that referenced this pull request May 20, 2026
## Why
Addresses post merge feedback from
#2399 (comment)
## Tests
N/A
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@shreyas-goenka@pietern@andrewnester