Skip to content

[v17] feat(join-tokens): [44794] Add Join Token form for GitHub#54477

Merged
nicholasmarais1158 merged 1 commit intobranch/v17from
nickmarais/backport-54308-branch/v17
May 12, 2025
Merged

[v17] feat(join-tokens): [44794] Add Join Token form for GitHub#54477
nicholasmarais1158 merged 1 commit intobranch/v17from
nickmarais/backport-54308-branch/v17

Conversation

@nicholasmarais1158
Copy link
Copy Markdown
Contributor

Backport #54308 to branch/v17

changelog: Create and edit GitHub join tokens from the Join Tokens page

* Fix/silence lint issues

* Fix spacing on side panel

* Return github-specific config from `GET /webapi/tokens`

* Add github form for create/edit

* Handle unsupported fields during edit

* Add js docs for check functions

* Add js docs for github check function

* Fix casing in test mocks

* Fix missing `readonly` prop

* Remove unused import

* Retain metadata (inc expiry) on token edit

* Remove mutual-exclusivity on repo/owner fields

* Improve supported fields readability

* Hide GHES config

* Ignore token not found errors when creating

* Lock GHES fields when using OSS

* Lint fix

* Return github-specific config from `GET /webapi/tokens`

* Support "token" in `/webapi/yaml/parse/:kind`

* Use empty time.Time for token expiry (`POST /webapi/tokens`)

* Revert createTokenForDiscoveryHandle changes

* Fix acronym casing

* Cover enterprise token types in tests

* Cover github tokens in existing tests

* Tweak handling of `tokenId`

* Check expiry is not overwritten

* Revert removing tokenId check

* Refactor supported fields

* Remove use of `X-Teleport-TokenName`
@nicholasmarais1158 nicholasmarais1158 added this pull request to the merge queue May 12, 2025
Merged via the queue into branch/v17 with commit 69d3e98 May 12, 2025
41 checks passed
@nicholasmarais1158 nicholasmarais1158 deleted the nickmarais/backport-54308-branch/v17 branch May 12, 2025 09:23
@doggydogworld doggydogworld mentioned this pull request Jun 3, 2025
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