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
[v13] Add Headless Polling to Teleport Connect #28975
[v13] Add Headless Polling to Teleport Connect #28975
Conversation
* Add WatchPendingHeadlessAuthentications rpc.
* Fix headless authentication matching logic for watcher and add test. * Move hasWatchPermissionForKind to a separate function. * Clean up hasWatchPermissionForKind. * Cleanup test code with suggestions from review.
* - Move tshd events client into the daemon service. - Replace gatway cert reissuer with a more reusable retryWithRelogin method. * Resolve comments.
* Implement headless watcher backend for Teleport Connect. * Add headless watcher to tshd daemon service. * Add SendPendingHeadlessAuthentication rpc to tshd events service. * Add UpdateHeadlessAuthenticationState rpc to the daemon service. * Address comments.
* Reduce headless watcher max backoff period to 90s; Propogate watcher error properly; Don't retry on not implemented error. * Stop watcher if it wasn't stopped already.
5cd5862
to
01764e4
Compare
…ake timeout less aggressive. (#29880)
@Joerger - this PR will require admin approval to merge due to its size. Consider breaking it up into a series smaller changes. |
Did you run into many conflicts when backporting this? If there were any in lib/teleterm and web/packages/teleterm, I could take a quick look to double check them. |
Nothing that was not easy to resolve. I reviewed the code and re-tested to make sure it still works. |
Backport Headless Polling PRs to branch/v13:
WatchPendingHeadlessAuthentications
rpc for headless polling #28234TestImportantModalSemaphore
#29880