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

Only show SSO Login on the login page. #2789

Closed
benarent opened this issue Jun 21, 2019 · 14 comments
Closed

Only show SSO Login on the login page. #2789

benarent opened this issue Jun 21, 2019 · 14 comments
Assignees
Labels
feature-request Used for new features in Teleport, improvements to current should be #enhancements sso Used for single sign on related tasks. ui

Comments

@benarent
Copy link
Contributor

Problem
Teleport lets admins quickly create users using tctl users add. This can be very handy when setting up the cluster, or adding a few Ops Admins. Once the team grows and Teleport is rolled out company wide it's preferred to login with an SSO provider. This results in a range of options for users, and for many they won't have a username, so it can be confusing.

image

Proposal

  1. Provide an option to turn off local login and just use SSO provider
  2. Provide an admin URL for companies that have a mix of SSH users and admins using local 2FA.
  3. Allow a set hierarchy of login options.
@benarent benarent added the feature-request Used for new features in Teleport, improvements to current should be #enhancements label Jun 21, 2019
@russjones
Copy link
Contributor

@benarent (1) is already available, set local_auth: false in file configuration. See the following PR: #2575

For (2), when hitting the endpoint to fetch the login page, we can check if local auth is disabled and the number of local users. If local auth is disabled or no local users exist, we can show only local login. If local auth is not disabled and local users exist, we can show the current login page.

What do you think?

@alex-kovoy
Copy link
Contributor

if local_auth is set to false why would we need to check for local users as well to hide these input fields?

@russjones
Copy link
Contributor

@alex-kovoy If local auth is disabled or no users exists (local auth being on in this case).

@dadayoo
Copy link

dadayoo commented Jun 27, 2019

@benarent (1) is already available, set local_auth: false in file configuration. See the following PR: #2575

For (2), when hitting the endpoint to fetch the login page, we can check if local auth is disabled and the number of local users. If local auth is disabled or no local users exist, we can show only local login. If local auth is not disabled and local users exist, we can show the current login page.

What do you think?

where we set local_auth: false? i'm running teleport v3.2

@robertwenquan
Copy link

tried to set local_auth: false under auth_service but seems it's not working after the service restart.

I am with Enterprise v4.0.0

# teleport version
Teleport Enterprise v4.0.0git:v4.0.0-0-gc7f55ac3 go1.12.1

@dadayoo
Copy link

dadayoo commented Jul 10, 2019

tried to set local_auth: false under auth_service but seems it's not working after the service restart.

I am with Enterprise v4.0.0

# teleport version
Teleport Enterprise v4.0.0git:v4.0.0-0-gc7f55ac3 go1.12.1

emm..not working at v3.2

@alex-kovoy
Copy link
Contributor

This feature request has not been implemented yet, so it's not currently possible to hide username/password input fields from web UI. So even if local auth is disabled on the server, the web UI will show these fields.

@robertwenquan
Copy link

Thanks @alex-kovoy . That solves the myth.

@benarent benarent added the ui label Jul 11, 2019
@benarent benarent added the sso Used for single sign on related tasks. label Aug 27, 2019
@webvictim
Copy link
Contributor

webvictim commented Oct 3, 2019

FYI for anyone who discovers this, the syntax for the local_auth setting is:

auth_service:
  authentication:
    local_auth: false

As noted in other comments, however, it doesn't currently disable display of the login boxes - it just disables the ability to create and log in with local users.

@webvictim
Copy link
Contributor

Another request for this feature today.

@benarent
Copy link
Contributor Author

We've a lot of changes coming in 4.2 but since we've just completely rebuilt the UI this should be easier for us to execute on. I'm going to put this into the 4.2 milestone.

@benarent benarent added this to the 4.2 "Alameda" milestone Oct 18, 2019
@benarent benarent modified the milestones: 4.2 "Alameda", 4.3 "Concord" Nov 1, 2019
ollyg added a commit to ollyg/teleport that referenced this issue Dec 20, 2019
ollyg added a commit to ollyg/teleport that referenced this issue Dec 20, 2019
ollyg added a commit to ollyg/teleport that referenced this issue Dec 20, 2019
This reverts commit 7f67571200846d345cd1aafb7d55a028f0f3693c.
ollyg added a commit to ollyg/teleport that referenced this issue Dec 20, 2019
This reverts commit 7f994f0bf6667dbc70d24773357e9e141991b0f9.
ollyg added a commit to ollyg/teleport that referenced this issue Dec 21, 2019
ollyg added a commit to ollyg/teleport that referenced this issue Dec 21, 2019
@benarent
Copy link
Contributor Author

We've another use case for gravitational/gravity#1006, in which a customer simply wants to

I would like to be able to hide user/pw fields as well, while still allowing "robot" users to log in via tele login

@alex-kovoy alex-kovoy assigned kimlisa and unassigned alex-kovoy Mar 3, 2020
@webvictim
Copy link
Contributor

This issue is closed, but the feature hasn't been rolled out. Is this specifically targetted to Teleport 5.0 only and we won't be adding it beforehand?

@kimlisa kimlisa added the hornet label Mar 26, 2020
@kimlisa
Copy link
Contributor

kimlisa commented Mar 26, 2020

@webvictim yes, the PR got merged against hornet branch which is teleport v5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Used for new features in Teleport, improvements to current should be #enhancements sso Used for single sign on related tasks. ui
Projects
None yet
Development

No branches or pull requests

7 participants