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

[FEATURE] Pass full username to session #6760

Open
3 of 4 tasks
blide opened this issue Oct 20, 2024 · 1 comment
Open
3 of 4 tasks

[FEATURE] Pass full username to session #6760

blide opened this issue Oct 20, 2024 · 1 comment
Labels

Comments

@blide
Copy link

blide commented Oct 20, 2024

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

Describe the feature

Currently username passed to the session is truncated up to first / or @. User logged in as bob/DOMAIN will produce session with user=bob.

ServiceUtils.getShortName(authFactory.getRemoteUser.getOrElse(req.getUsername))

Therefore there is no way to differentiate users bob/DOMAIN1 and bob/DOMAIN2 inside session.
Issue suggests to pass domain part to the session.

Motivation

When using custom GroupProvider and SessionConfAdvisor it might required to know user's domain part of username.
In my case it's possible there are bob/DOMAIN1 and bob/DOMAIN2, so I want to differentiate them.

Describe the solution

Add full username including a domain part to the session config, e.g. kyuubi.session.full.user field.

Additional context

No response

Are you willing to submit PR?

  • Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
  • No. I cannot submit a PR at this time.
Copy link

Hello @blide,
Thanks for finding the time to report the issue!
We really appreciate the community's efforts to improve Apache Kyuubi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant