Skip to content

[v17] Show only allowed logins for SSH servers in Connect#59068

Merged
gzdunek merged 5 commits intobranch/v17from
gzdunek/backport-59026/v17
Sep 16, 2025
Merged

[v17] Show only allowed logins for SSH servers in Connect#59068
gzdunek merged 5 commits intobranch/v17from
gzdunek/backport-59026/v17

Conversation

@gzdunek
Copy link
Copy Markdown
Contributor

@gzdunek gzdunek commented Sep 12, 2025

Backport #59026 to branch/v17

changelog: When selecting a login for an SSH server, Teleport Connect now shows only logins allowed by RBAC for that specific server rather than showing all logins which the user has access to

* Move `calculateSSHLogins` to `lib/client`

* Calculate allowed logins in tshd

* Take logins from resource, not `loggedInUser`

* Remove `pipe`

* Remove `sshLogins`

* Lint

(cherry picked from commit f743d25)
@gzdunek gzdunek enabled auto-merge September 12, 2025 16:35
@gzdunek gzdunek added this pull request to the merge queue Sep 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 12, 2025
@gzdunek gzdunek added this pull request to the merge queue Sep 15, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 15, 2025
@gzdunek gzdunek enabled auto-merge September 15, 2025 08:31
@gzdunek gzdunek added this pull request to the merge queue Sep 15, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 15, 2025
@gzdunek gzdunek enabled auto-merge September 16, 2025 07:36
@gzdunek gzdunek added this pull request to the merge queue Sep 16, 2025
Merged via the queue into branch/v17 with commit cf775ca Sep 16, 2025
44 checks passed
@gzdunek gzdunek deleted the gzdunek/backport-59026/v17 branch September 16, 2025 08:14
@doggydogworld doggydogworld mentioned this pull request Sep 18, 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