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

Wire touch ID login and registration to tsh #12475

Merged
merged 4 commits into from
May 9, 2022

Conversation

codingllama
Copy link
Contributor

Add the --mfa-mode flag and wire touch ID into tsh authn/registration.

In the default MFA mode (auto), touch ID should be eager if there is a credential present in the system. Users may change this behavior using other MFA modes.

See https://github.com/gravitational/teleport/blob/master/rfd/0054-passwordless-macos.md.

#9160

@github-actions github-actions bot requested review from lxea and nklaassen May 5, 2022 22:31
@github-actions github-actions bot added the tsh tsh - Teleport's command line tool for logging into nodes running Teleport. label May 5, 2022
@codingllama
Copy link
Contributor Author

Branched from #12436, target switches to master automatically once it lands.

@codingllama codingllama force-pushed the codingllama/touchid-register branch from a16d082 to 8d17020 Compare May 6, 2022 14:32
@codingllama codingllama force-pushed the codingllama/touchid-tsh branch from 4dd02a4 to 3480211 Compare May 6, 2022 15:11
@codingllama codingllama force-pushed the codingllama/touchid-register branch from e587877 to c0d054f Compare May 6, 2022 18:10
@codingllama codingllama force-pushed the codingllama/touchid-tsh branch from 3480211 to e9af7b0 Compare May 6, 2022 18:10
@codingllama
Copy link
Contributor Author

Friendly ping @lxea.

@codingllama codingllama force-pushed the codingllama/touchid-register branch from c0d054f to 8148bdf Compare May 9, 2022 12:30
@codingllama codingllama force-pushed the codingllama/touchid-tsh branch from e9af7b0 to a1ea67b Compare May 9, 2022 12:30
Base automatically changed from codingllama/touchid-register to master May 9, 2022 13:21
@codingllama codingllama force-pushed the codingllama/touchid-tsh branch from a1ea67b to 12867fa Compare May 9, 2022 13:22
@codingllama codingllama enabled auto-merge (squash) May 9, 2022 13:23
@codingllama codingllama merged commit 6839291 into master May 9, 2022
@codingllama codingllama deleted the codingllama/touchid-tsh branch May 9, 2022 14:46
codingllama added a commit that referenced this pull request Jun 23, 2022
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 added a commit that referenced this pull request Jun 24, 2022
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 added a commit that referenced this pull request Jun 24, 2022
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 added a commit that referenced this pull request Jun 24, 2022
Pass proxy address to PromptMFAChallenge calls (#13772)

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 added a commit that referenced this pull request Jun 24, 2022
Pass proxy address to PromptMFAChallenge calls (#13772)

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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