Skip to content

Adding tctl scoped tokens family of subcommands#60035

Merged
eriktate merged 1 commit intomasterfrom
eriktate/tctl-scoped-tokens
Oct 27, 2025
Merged

Adding tctl scoped tokens family of subcommands#60035
eriktate merged 1 commit intomasterfrom
eriktate/tctl-scoped-tokens

Conversation

@eriktate
Copy link
Copy Markdown
Contributor

@eriktate eriktate commented Oct 8, 2025

This PR adds the following subcommands to tctl:

tctl scoped tokens add
tctl scoped tokens ls
tctl scoped tokens del

It largely mimics the existing tctl tokens ... subcommands while adding new fields for assigning scopes or filtering by scopes.

@github-actions github-actions bot requested review from greedy52 and kiosion October 8, 2025 01:15
@github-actions github-actions bot added size/lg tctl tctl - Teleport admin tool labels Oct 8, 2025
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from cb28a3e to d960a90 Compare October 8, 2025 01:23
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from ceb6d07 to 3360767 Compare October 8, 2025 13:21
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from 7849329 to 703c043 Compare October 10, 2025 19:30
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch 2 times, most recently from ad2ff69 to 1810774 Compare October 14, 2025 18:22
Copy link
Copy Markdown
Contributor

@fspmarshall fspmarshall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My general feedback would be that brevity is the soul of wit. There's a lot here that just isn't necessary yet, and a lot of bridges I'd rather cross when we come to them. Can we pair this down to the essentials (ttl, role, etc)?

@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from b33fac5 to 9fb4699 Compare October 16, 2025 03:01
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from 1810774 to e465ec9 Compare October 16, 2025 03:01
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch 3 times, most recently from 5f9bf01 to 735c60e Compare October 16, 2025 16:03
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch 3 times, most recently from 2b17244 to 39d932c Compare October 17, 2025 14:23
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from a1dbcc3 to 7846fe2 Compare October 20, 2025 19:47
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from 39d932c to e20ed0e Compare October 20, 2025 19:47
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from 7846fe2 to 0ee1260 Compare October 20, 2025 20:10
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch 6 times, most recently from a06ffaa to b4d17ad Compare October 21, 2025 16:49
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from b4d17ad to 3d2df71 Compare October 21, 2025 17:53
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from 690009e to 4b2557b Compare October 21, 2025 20:50
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from 3d2df71 to 53825d3 Compare October 21, 2025 20:51
@eriktate eriktate force-pushed the eriktate/scoped-join-tokens-backend branch from 4b2557b to e810e5a Compare October 22, 2025 03:52
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from 53825d3 to f2d60b0 Compare October 22, 2025 03:59
Base automatically changed from eriktate/scoped-join-tokens-backend to master October 22, 2025 04:35
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from f2d60b0 to 6662596 Compare October 22, 2025 13:12
@rosstimothy rosstimothy requested a review from nklaassen October 24, 2025 13:13
@eriktate eriktate requested a review from nklaassen October 27, 2025 15:22
@eriktate eriktate force-pushed the eriktate/tctl-scoped-tokens branch from 3d97d66 to 73b2a30 Compare October 27, 2025 16:45
@eriktate eriktate added the no-changelog Indicates that a PR does not require a changelog entry label Oct 27, 2025
@eriktate eriktate added this pull request to the merge queue Oct 27, 2025
Merged via the queue into master with commit 318fad1 Oct 27, 2025
42 of 43 checks passed
@eriktate eriktate deleted the eriktate/tctl-scoped-tokens branch October 27, 2025 18:36
eriktate added a commit that referenced this pull request Oct 29, 2025
fspmarshall pushed a commit that referenced this pull request Oct 31, 2025
eriktate added a commit that referenced this pull request Nov 4, 2025
mmcallister pushed a commit that referenced this pull request Nov 6, 2025
mmcallister pushed a commit that referenced this pull request Nov 19, 2025
mmcallister pushed a commit that referenced this pull request Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/lg tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants