This is an automated email from the ASF dual-hosted git repository.
bowenliang pushed a commit to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/branch-1.7 by this push:
new c439f5abd [KYUUBI #5187] [Minor] Fix info message in UserFilter
c439f5abd is described below
commit c439f5abd1cf8154acbe17a1121b4e04955f260d
Author: liangbowen <[email protected]>
AuthorDate: Tue Aug 22 13:03:34 2023 +0800
[KYUUBI #5187] [Minor] Fix info message in UserFilter
### _Why are the changes needed?_
- fix info message with correct string placeholder of user in UserFilter
### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including
negative and positive cases if possible
- [ ] Add screenshots for manual tests if appropriate
- [x] [Run
test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests)
locally before make a pull request
### _Was this patch authored or co-authored using generative AI tooling?_
No.
Closes #5187 from bowenliang123/userfilter-str.
Closes #5187
25d370ee3 [liangbowen] fix info message in UserFilter
Authored-by: liangbowen <[email protected]>
Signed-off-by: liangbowen <[email protected]>
(cherry picked from commit 739e1a61a7e6ea394c6a297752e5198c38289110)
Signed-off-by: liangbowen <[email protected]>
---
.../apache/kyuubi/service/authentication/ldap/UserFilterFactory.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/kyuubi-common/src/main/scala/org/apache/kyuubi/service/authentication/ldap/UserFilterFactory.scala
b/kyuubi-common/src/main/scala/org/apache/kyuubi/service/authentication/ldap/UserFilterFactory.scala
index 7c2f22ed8..d499c67ef 100644
---
a/kyuubi-common/src/main/scala/org/apache/kyuubi/service/authentication/ldap/UserFilterFactory.scala
+++
b/kyuubi-common/src/main/scala/org/apache/kyuubi/service/authentication/ldap/UserFilterFactory.scala
@@ -35,7 +35,7 @@ class UserFilter(_userFilter: Seq[String]) extends Filter
with Logging {
@throws[AuthenticationException]
override def apply(ldap: DirSearch, user: String): Unit = {
- info("Authenticating user '$user' using user filter")
+ info(s"Authenticating user '$user' using user filter")
val userName = LdapUtils.extractUserName(user).toLowerCase
if (!userFilter.contains(userName)) {
info("Authentication failed based on user membership")
