-
Notifications
You must be signed in to change notification settings - Fork 868
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
[Bug] kyuubiClientTicketCache
always use same authentication
#6396
Labels
Comments
4 tasks
cxzl25
pushed a commit
that referenced
this issue
May 24, 2024
…he key This pull request fixes #6396 By using a cache to store CachingKerberosAuthentication objects keyed by the ticket cache path, we ensure that each unique ticket cache path generates a distinct authentication object. - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) --- - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6401 from lsm1/branch-kyuubi-6396. Closes #6396 bb8f738 [senmiaoliu] fix kyuubiClientTicketCache Authored-by: senmiaoliu <senmiaoliu@trip.com> Signed-off-by: Shaoyun Chen <csy@apache.org> (cherry picked from commit 4285abc)
pan3793
pushed a commit
that referenced
this issue
May 27, 2024
# 🔍 Description ## Issue References 🔗 This pull request fixes #6396 ## Describe Your Solution 🔧 NPE will occur when kyuubiClientTicketCache is not specified. `ConcurrentHashMap` does not allow null key. ## Types of changes 🔖 - [X] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6420 from wForget/hotfix2. Closes #6396 e5d5f9d [wforget] address comment fe9ecd1 [wforget] Avoid NPE Authored-by: wforget <643348094@qq.com> Signed-off-by: Cheng Pan <chengpan@apache.org>
pan3793
pushed a commit
that referenced
this issue
May 27, 2024
# 🔍 Description ## Issue References 🔗 This pull request fixes #6396 ## Describe Your Solution 🔧 NPE will occur when kyuubiClientTicketCache is not specified. `ConcurrentHashMap` does not allow null key. ## Types of changes 🔖 - [X] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6420 from wForget/hotfix2. Closes #6396 e5d5f9d [wforget] address comment fe9ecd1 [wforget] Avoid NPE Authored-by: wforget <643348094@qq.com> Signed-off-by: Cheng Pan <chengpan@apache.org> (cherry picked from commit e4ecde2) Signed-off-by: Cheng Pan <chengpan@apache.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Code of Conduct
Search before asking
Describe the bug
From 1.8 onwards,
kyuubiClientTicketCache
can be used for Kerberos authentication. However, if an incorrect path is specified initially, subsequent attempts to authenticate using the correct path will fail.Affects Version(s)
master
Kyuubi Server Log Output
No response
Kyuubi Engine Log Output
No response
Kyuubi Server Configurations
No response
Kyuubi Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?
The text was updated successfully, but these errors were encountered: