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

[v16] Support hardware keys prompts in Connect #49701

Merged
merged 4 commits into from
Dec 4, 2024

Conversation

gzdunek
Copy link
Contributor

@gzdunek gzdunek commented Dec 3, 2024

Backport #47652 to branch/v16

changelog: Add support for hardware keys in Teleport Connect

* Add new protos for hardware key prompts

* Implement hardware key prompts on the daemon side

* Show prompts in the UI

* `PromptHardwareKeyPINAsk` -> `PromptHardwareKeyPIN`

* Improve proto docs

* `PromptHardwareKeySlotOverwrite` -> `ConfirmHardwareKeySlotOverwrite`

* Fix typo

* Remove unnecessary `form`

* Pass an enum to `AskPIN` instead of the entire message

* Remove an invalid restriction of PIN/PUK to numbers only

* Improve the copy

(cherry picked from commit 22b5014)
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-49701.d212ksyjt6y4yg.amplifyapp.com

@gzdunek gzdunek requested a review from avatus December 4, 2024 10:20
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from Joerger December 4, 2024 15:23
# Conflicts:
#	gen/proto/go/teleport/lib/teleterm/v1/tshd_events_service.pb.go
@gzdunek gzdunek enabled auto-merge December 4, 2024 16:29
@gzdunek gzdunek added this pull request to the merge queue Dec 4, 2024
Merged via the queue into branch/v16 with commit 56fd464 Dec 4, 2024
45 checks passed
@gzdunek gzdunek deleted the gzdunek/backport-47652-v16 branch December 4, 2024 17:22
@camscale camscale mentioned this pull request Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants