Skip to content

Conversation

@Joerger
Copy link
Contributor

@Joerger Joerger commented Apr 4, 2025

Part of RFD 199

Ensure that a single hardware key service, and client store, is used per client process. This ensures that new hardware key service features (hardware key agent, pin caching, yubikey connection caching) works properly and that the client is not competing with itself (in different goroutines) for resources.

Caveat: This change only applies to tsh, tctl, and Teleport Connect. tbot, api/client, and integrations need additional work.

TODO: move some of these changes out to separate PRs

Depends on #53703

@Joerger Joerger force-pushed the joerger/key-info branch from 8a52b06 to 40e294e Compare April 7, 2025 23:55
@Joerger Joerger force-pushed the joerger/init-hardware-key-service branch from 6a8e10b to a785f31 Compare April 8, 2025 00:39
@Joerger Joerger force-pushed the joerger/key-info branch from 40e294e to 1ba6865 Compare April 8, 2025 17:56
@Joerger Joerger force-pushed the joerger/key-info branch from 1ba6865 to 07aaedc Compare April 9, 2025 23:07
@Joerger Joerger force-pushed the joerger/init-hardware-key-service branch from a785f31 to 983c196 Compare April 9, 2025 23:17
@Joerger Joerger force-pushed the joerger/key-info branch from 07aaedc to 9d652e2 Compare April 9, 2025 23:44
@Joerger Joerger force-pushed the joerger/init-hardware-key-service branch from 983c196 to dcc1c2d Compare April 9, 2025 23:50
@Joerger Joerger force-pushed the joerger/key-info branch 2 times, most recently from b68108f to 0931e80 Compare April 10, 2025 00:32
@Joerger Joerger force-pushed the joerger/init-hardware-key-service branch from dcc1c2d to f944298 Compare April 10, 2025 01:36
@Joerger Joerger force-pushed the joerger/key-info branch 3 times, most recently from 32edb34 to 35156bf Compare April 11, 2025 16:35
Base automatically changed from joerger/key-info to master April 11, 2025 17:15
Joerger added 2 commits April 11, 2025 16:30
…s so that the default value (false) results in the desired default behavior.
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.

2 participants