Skip to content
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.

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

feat: support referencing secret in any field of oidc config #13475

Merged
merged 1 commit into from
May 28, 2023

Conversation

alexmt
Copy link
Collaborator

@alexmt alexmt commented May 5, 2023

Closes #9680

PR allows referencing secret values in any field of OIDC config.

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@alexmt alexmt requested review from jessesuen and gdsoumya May 5, 2023 21:29
@codecov
Copy link

codecov bot commented May 5, 2023

Codecov Report

Patch coverage: 70.83% and project coverage change: -0.02 ⚠️

Comparison is base (17d0aca) 49.14% compared to head (b29f44a) 49.13%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #13475      +/-   ##
==========================================
- Coverage   49.14%   49.13%   -0.02%     
==========================================
  Files         248      248              
  Lines       42893    42903      +10     
==========================================
  Hits        21079    21079              
- Misses      19695    19703       +8     
- Partials     2119     2121       +2     
Impacted Files Coverage Δ
util/settings/settings.go 49.69% <65.00%> (+0.32%) ⬆️
util/dex/config.go 85.85% <100.00%> (+0.25%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@gdsoumya gdsoumya left a comment

Choose a reason for hiding this comment

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

LGTM!

@crenshaw-dev crenshaw-dev merged commit 47554e4 into argoproj:master May 28, 2023
24 checks passed
yyzxw pushed a commit to yyzxw/argo-cd that referenced this pull request Aug 9, 2023
…j#13475)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
tesla59 pushed a commit to tesla59/argo-cd that referenced this pull request Dec 16, 2023
…j#13475)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Sign up for free to 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.

cliClientID doesn't get interpolated from secrets
3 participants