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

adding oidc desired username as option in config #1997

Closed
wants to merge 3 commits into from

Conversation

Lite5h4dow
Copy link

@Lite5h4dow Lite5h4dow commented Jun 27, 2024

This PR allows people to use their desired username from their oidc provider instead of their email address. this is a user-facing feature and doesn't change any underlying processes

  • have read the CONTRIBUTING.md file
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

@kradalby
Copy link
Collaborator

Hi, please fix the last checkboxes in the pr and add a justification for the feature, it seems like a linked issue is also missing

@Lite5h4dow
Copy link
Author

Lite5h4dow commented Jun 28, 2024

related issues:

I did discuss this In the discord last year some time, and again earlier this week, but decided to just make a pr.

@Lite5h4dow
Copy link
Author

This PR makes a simple adjustment to the username discernment process when using OIDC, we already pulled in the desired_username from the token, we just needed to clean it up and use it.

@Lite5h4dow
Copy link
Author

is it just me or is there currently no support for profile images? if not that might be what I add next. i just don't know where we would need to put it so the tailscale client could pick it up (this would only really be a feature for android, windows and mac clients, I don't believe the linux client has a front end, its all cli.

@ChibangLW
Copy link
Contributor

Maybe have a look at #1934. It is a more general improvement which also results in usable preferred_username.

As well as #1981, which adds profile picture support.

I am not sure whats the maintainers stand on these PRs and in general on additional OIDC features.

@Lite5h4dow
Copy link
Author

well if your gonna support OIDC, why half arse it?

@SirBomble SirBomble mentioned this pull request Jul 2, 2024
2 tasks
@kradalby kradalby added the OIDC OpenID Connect related issues label Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OIDC OpenID Connect related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants