Repository: sqoop Updated Branches: refs/heads/branch-1.99.6 8eee6f52b -> 0c06834db
SQOOP-2278: Sqoop2: Fix bug for all user could read connector (Richard Zhou via Abraham Elmahrek) Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/0c06834d Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/0c06834d Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/0c06834d Branch: refs/heads/branch-1.99.6 Commit: 0c06834dba866baf03ac4797eac559a2bc12b06b Parents: 8eee6f5 Author: Abraham Elmahrek <[email protected]> Authored: Thu Apr 2 12:41:54 2015 -0700 Committer: Abraham Elmahrek <[email protected]> Committed: Thu Apr 2 12:44:52 2015 -0700 ---------------------------------------------------------------------- .../sqoop/security/Authorization/AuthorizationEngine.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sqoop/blob/0c06834d/security/src/main/java/org/apache/sqoop/security/Authorization/AuthorizationEngine.java ---------------------------------------------------------------------- diff --git a/security/src/main/java/org/apache/sqoop/security/Authorization/AuthorizationEngine.java b/security/src/main/java/org/apache/sqoop/security/Authorization/AuthorizationEngine.java index 8c718e3..e6bf8be 100644 --- a/security/src/main/java/org/apache/sqoop/security/Authorization/AuthorizationEngine.java +++ b/security/src/main/java/org/apache/sqoop/security/Authorization/AuthorizationEngine.java @@ -181,12 +181,13 @@ public class AuthorizationEngine { if (!user_name.equals(link.getCreationUser())) { privilegesNeedCheck.add(privilege); } - } - if (MResource.TYPE.JOB.name().equalsIgnoreCase(privilege.getResource().getType())) { + } else if (MResource.TYPE.JOB.name().equalsIgnoreCase(privilege.getResource().getType())) { MJob job = repository.findJob(Long.valueOf(privilege.getResource().getName())); if (!user_name.equals(job.getCreationUser())) { privilegesNeedCheck.add(privilege); } + } else { + privilegesNeedCheck.add(privilege); } }
