[
https://issues.apache.org/jira/browse/KNOX-3497?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18121037#comment-18121037
]
ASF subversion and git services commented on KNOX-3497:
-------------------------------------------------------
Commit 7dfa327380a71ab16e33f25222bf20efd52e8de5 in knox's branch
refs/heads/master from Sandor Molnar
[ https://gitbox.apache.org/repos/asf?p=knox.git;h=7dfa32738 ]
KNOX-3497: Resolve credential-store aliases in LDAP proxy backend password
properties (#1436)
> LdapProxyBackend does not resolve credential-store aliases for bind/system
> password
> -----------------------------------------------------------------------------------
>
> Key: KNOX-3497
> URL: https://issues.apache.org/jira/browse/KNOX-3497
> Project: Apache Knox
> Issue Type: Improvement
> Components: Server
> Affects Versions: 3.0.0
> Reporter: Sandor Molnar
> Assignee: Sandor Molnar
> Priority: Major
> Fix For: 3.1.0
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> {{LdapProxyBackend}} reads the AD service-account password as a literal
> string and never resolves credential-store aliases:
> * {{LdapProxyBackend.java:158-161}} - {{bindPassword =
> config.get("bindPassword") / config.get("systemPassword");}} value passed
> verbatim to {{setCredentials()}} (line 282).
> * No {{AliasService}} reference anywhere in the class or its factory.
> * Upstream config ({{{}GatewayConfigImpl.getLDAPInterceptorConfig{}}}) is a
> plain prefix scan with no alias expansion; the only alias-aware LDAP calls
> ({{{}KnoxLDAPServerManager.start{}}}, {{{}resolveSslKeystorePassword{}}}) are
> off this path.
> {*}Impact{*}: An alias reference in
> {{gateway.ldap.interceptor.<name>.bindPassword/.systemPassword}} is used as
> the literal password, so the AD bind fails. Passwords can only be stored in
> cleartext config today.
> {*}Fix{*}: Resolve aliases for the bind/system password on the backend path
> (in {{LdapProxyBackend.init()}} or when the interceptor config is assembled),
> preserving literal values for back-compat.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
