You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by "Krzysztof Adamski (Jira)" <ji...@apache.org> on 2019/12/19 07:14:00 UTC
[jira] [Issue Comment Deleted] (RANGER-2678) RangerUgSyncRESTClient
sets password as username for basic auth
[ https://issues.apache.org/jira/browse/RANGER-2678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Krzysztof Adamski updated RANGER-2678:
--------------------------------------
Comment: was deleted
(was: diff --git a/ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.java b/ugsync/src/main/java/org/apache/ranger/unixusersync/procdiff --git a/ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.java b/ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.javaindex e4024a25a..991d71f00 100644--- a/ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.java+++ b/ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.java@@ -80,7 +80,7 @@ public class RangerUgSyncRESTClient extends RangerRESTClient {
setClient(Client.create(config)); if (StringUtils.isNotEmpty(getUsername()) && StringUtils.isNotEmpty(getPassword())) \{- getClient().addFilter(new HTTPBasicAuthFilter(getPassword(), getPassword()));+ getClient().addFilter(new HTTPBasicAuthFilter(getUsername(), getPassword())); } } })
> RangerUgSyncRESTClient sets password as username for basic auth
> ---------------------------------------------------------------
>
> Key: RANGER-2678
> URL: https://issues.apache.org/jira/browse/RANGER-2678
> Project: Ranger
> Issue Type: Bug
> Components: usersync
> Affects Versions: master
> Reporter: Krzysztof Adamski
> Priority: Major
>
> RANGER-2591 introduced changes to usersync (unix/ldap).
> The following line is however broken
> [https://github.com/apache/ranger/blame/master/ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.java#L83]
> getClient().addFilter(new HTTPBasicAuthFilter(getPassword(), getPassword()));
> it should be
> getClient().addFilter(new HTTPBasicAuthFilter(getUsername(), getPassword()));
--
This message was sent by Atlassian Jira
(v8.3.4#803005)