Skip to content

MWI: Add audit events for bound keypair joining #55701

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

Merged
merged 6 commits into from
Jun 24, 2025

Conversation

timothyb89
Copy link
Contributor

@timothyb89 timothyb89 commented Jun 13, 2025

This adds 3 new audit events for bound keypair joining:

  • join_token.bound_keypair.recovery - emitted when a join triggers a recovery (first join, or join with expired certs)
  • join_token.bound_keypair.rotation - emitted when a keypair rotation takes place
  • join_token.bound_keypair.join_state_verification_failed - emitted when the client provides an invalid join state document

This also cleans up and obscures various error messages returned to clients since we now log more specific explanations for why a recovery/rotation/verification failed.

@timothyb89 timothyb89 force-pushed the timothyb89/bound-keypair-enforce-generation-counter branch from 492e734 to e1e3c19 Compare June 17, 2025 02:12
Base automatically changed from timothyb89/bound-keypair-enforce-generation-counter to master June 17, 2025 17:30
@timothyb89 timothyb89 force-pushed the timothyb89/bound-keypair-audit-events branch from 5b457ab to 106c0df Compare June 18, 2025 00:36
@timothyb89 timothyb89 added no-changelog Indicates that a PR does not require a changelog entry labels Jun 18, 2025
@timothyb89 timothyb89 marked this pull request as ready for review June 20, 2025 17:54
@github-actions github-actions bot requested review from gzdunek and ryanclark June 20, 2025 17:55
@public-teleport-github-review-bot

@timothyb89 - this PR will require admin approval to merge due to its size. Consider breaking it up into a series smaller changes.

@github-actions github-actions bot added audit-log Issues related to Teleports Audit Log size/xl ui labels Jun 20, 2025
This adds 3 new audit events for bound keypair joining:
- `join_token.bound_keypair.recovery` - emitted when a join triggers
  a recovery (first join, or join with expired certs)
- `join_token.bound_keypair.rotation` - emitted when a keypair
  rotation takes place
- `join_token.bound_keypair.join_state_verification_failed` - emitted
  when the client provides an invalid join state document
@timothyb89 timothyb89 force-pushed the timothyb89/bound-keypair-audit-events branch from 88e8e17 to 8731de0 Compare June 20, 2025 18:03
@timothyb89
Copy link
Contributor Author

review note: I removed the event schema update (gen/go/eventschema/zz_generated.eventschema.go) since it wildly increased the size of the PR and doesn't seem to be maintained.

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from ryanclark June 24, 2025 01:29
@timothyb89 timothyb89 enabled auto-merge June 24, 2025 01:30
@timothyb89 timothyb89 added this pull request to the merge queue Jun 24, 2025
Merged via the queue into master with commit 31c7278 Jun 24, 2025
45 checks passed
@timothyb89 timothyb89 deleted the timothyb89/bound-keypair-audit-events branch June 24, 2025 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audit-log Issues related to Teleports Audit Log machine-id no-changelog Indicates that a PR does not require a changelog entry size/xl ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants