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

[v10] Pass proxy address to PromptMFAChallenge calls (#13772) #13855

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

codingllama
Copy link
Contributor

Reinstates some logic that was removed on #12475 and changes optsOverride to a
function, so there is less ambiguity in dealing with booleans / default values.

  • Pass proxy address to PromptMFAChallenge calls
  • Add coverage for TeleportClient.PromptMFAChallenge

@github-actions github-actions bot requested review from ryanclark and smallinsky June 24, 2022 18:16
@github-actions github-actions bot added the tsh tsh - Teleport's command line tool for logging into nodes running Teleport. label Jun 24, 2022
@codingllama codingllama enabled auto-merge (squash) June 24, 2022 19:03
@codingllama codingllama force-pushed the codingllama/v10-proxy-prompt-mfa branch 2 times, most recently from e0775ad to 3b34796 Compare June 24, 2022 20:26
Reinstates some logic that was removed on #12475 and changes `optsOverride` to a
function, so there is less ambiguity in dealing with booleans / default values.

* Pass proxy address to PromptMFAChallenge calls
* Add coverage for TeleportClient.PromptMFAChallenge
@codingllama codingllama force-pushed the codingllama/v10-proxy-prompt-mfa branch from 3b34796 to e9b47ef Compare June 24, 2022 21:54
@codingllama codingllama merged commit f3de3b7 into branch/v10 Jun 24, 2022
@webvictim webvictim mentioned this pull request Jul 12, 2022
@codingllama codingllama deleted the codingllama/v10-proxy-prompt-mfa branch July 14, 2022 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport tsh tsh - Teleport's command line tool for logging into nodes running Teleport.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants