Skip to content
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

Teleport Connect not filtering SSH users like Web UI #23723

Open
stevenGravy opened this issue Mar 28, 2023 · 2 comments
Open

Teleport Connect not filtering SSH users like Web UI #23723

stevenGravy opened this issue Mar 28, 2023 · 2 comments
Labels
bug c-ecm Internal Customer Reference teleport-connect Issues related to Teleport Connect.

Comments

@stevenGravy
Copy link
Contributor

stevenGravy commented Mar 28, 2023

Expected behavior:

The list of users for SSH servers only include the applicable users

Current behavior:

All users can show when applying a Access Request.

Web UI

image

Same Access Request applied in Teleport Connect:

image

Bug details:

  • Teleport version: 12.1.2
  • Recreation steps

Create a Acess request to another role with different users.
Request and be granted that access request.
Apply within Teleport Connect.
Note that you will see both the original certificate login users and the access request applied cert logins.

Related issues:

@stevenGravy stevenGravy added bug teleport-connect Issues related to Teleport Connect. labels Mar 28, 2023
@stevenGravy stevenGravy changed the title Teleport Connect not filtering users like Web UI Teleport Connect not filtering SSH users like Web UI Mar 28, 2023
@pschisa pschisa added the c-ecm Internal Customer Reference label May 2, 2024
@ravicious
Copy link
Member

I just tried to address this while fixing pagination (#42501). It's entirely doable as it's mostly a matter of extracting calculateSSHLogins to lib/client and changing it so that it accepts localLogins []string instead of a tlsca.Identity.

However, constructing all arguments this function accepts and the dependencies of those arguments requires adding a bunch of new methods. It's like a day of work, but I just don't have the time for that at the moment.

@webvictim
Copy link
Contributor

+1 for this, just noticed it again on 16.1.7. Quite disconcerting - I thought I had an error in my role until I went and checked the web UI to compare.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug c-ecm Internal Customer Reference teleport-connect Issues related to Teleport Connect.
Projects
None yet
Development

No branches or pull requests

4 participants