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
exec credential provider: don't run exec plugin with basic auth #102175
exec credential provider: don't run exec plugin with basic auth #102175
Conversation
/triage accepted |
I would like to see a new test in |
cmd test added: b73b5547832578b8b98cec8168e11c1773307cbd note that this pr will start failing once #102152 merges, since a metrics test assertion from that pr is now incorrect. |
b73b554
to
859d87f
Compare
/lgtm |
If a user specifies basic auth, then apply the same short circuit logic that we do for bearer tokens (see comment). Signed-off-by: Andrew Keesler <akeesler@vmware.com>
859d87f
to
9dee2b9
Compare
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ankeesler, deads2k, enj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
If a user specifies basic auth, then apply the same short circuit logic
that we do for bearer tokens (see comment).
Signed-off-by: Andrew Keesler akeesler@vmware.com
What type of PR is this?
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
None.
Special notes for your reviewer:
Does this PR introduce a user-facing change?
/sig auth