You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by mp...@apache.org on 2015/11/20 20:49:05 UTC
[1/2] ambari git commit: AMBARI-13997. Ambari force-writing usernames
to be lowercase in DB. (mpapirkovskyy)
Repository: ambari
Updated Branches:
refs/heads/branch-2.1 2e96a5d5a -> 2fef316d5
refs/heads/trunk c6e946f05 -> 2d18d2df9
AMBARI-13997. Ambari force-writing usernames to be lowercase in DB. (mpapirkovskyy)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/2d18d2df
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/2d18d2df
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/2d18d2df
Branch: refs/heads/trunk
Commit: 2d18d2df97b7bb66f678cd959e329bf97e1e69ab
Parents: c6e946f
Author: Myroslav Papirkovskyi <mp...@hortonworks.com>
Authored: Fri Nov 20 21:47:55 2015 +0200
Committer: Myroslav Papirkovskyi <mp...@hortonworks.com>
Committed: Fri Nov 20 21:47:55 2015 +0200
----------------------------------------------------------------------
.../org/apache/ambari/server/api/services/UserService.java | 1 +
.../java/org/apache/ambari/server/controller/AmbariServer.java | 1 +
.../main/java/org/apache/ambari/server/orm/dao/UserDAO.java | 6 +++---
.../security/authorization/AmbariLocalUserDetailsService.java | 4 +++-
ambari-web/app/router.js | 2 +-
5 files changed, 9 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/2d18d2df/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java b/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
index 5abf4ed..fea5eca 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
@@ -31,6 +31,7 @@ import javax.ws.rs.core.UriInfo;
import org.apache.ambari.server.api.resources.ResourceInstance;
import org.apache.ambari.server.controller.spi.Resource;
+import org.apache.commons.lang.StringUtils;
import java.util.Collections;
http://git-wip-us.apache.org/repos/asf/ambari/blob/2d18d2df/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java b/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
index a3c5fcc..56034d9 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
@@ -81,6 +81,7 @@ import org.apache.ambari.server.orm.entities.MetainfoEntity;
import org.apache.ambari.server.resources.ResourceManager;
import org.apache.ambari.server.resources.api.rest.GetResource;
import org.apache.ambari.server.scheduler.ExecutionScheduleManager;
+import org.apache.ambari.server.security.AmbariEntryPoint;
import org.apache.ambari.server.security.AmbariServerSecurityHeaderFilter;
import org.apache.ambari.server.security.CertificateManager;
import org.apache.ambari.server.security.SecurityFilter;
http://git-wip-us.apache.org/repos/asf/ambari/blob/2d18d2df/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java b/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
index 1bd83f8..7b82cb3 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
@@ -145,21 +145,21 @@ public class UserDAO {
@Transactional
public void create(Set<UserEntity> users) {
for (UserEntity user: users) {
- user.setUserName(user.getUserName().toLowerCase());
+// user.setUserName(user.getUserName().toLowerCase());
entityManagerProvider.get().persist(user);
}
}
@Transactional
public UserEntity merge(UserEntity user) {
- user.setUserName(user.getUserName().toLowerCase());
+// user.setUserName(user.getUserName().toLowerCase());
return entityManagerProvider.get().merge(user);
}
@Transactional
public void merge(Set<UserEntity> users) {
for (UserEntity user: users) {
- user.setUserName(user.getUserName().toLowerCase());
+// user.setUserName(user.getUserName().toLowerCase());
entityManagerProvider.get().merge(user);
}
}
http://git-wip-us.apache.org/repos/asf/ambari/blob/2d18d2df/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java b/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
index 2aae8a0..5607cc5 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
@@ -27,6 +27,7 @@ import org.apache.ambari.server.orm.entities.MemberEntity;
import org.apache.ambari.server.orm.entities.PrincipalEntity;
import org.apache.ambari.server.orm.entities.PrivilegeEntity;
import org.apache.ambari.server.orm.entities.UserEntity;
+import org.apache.commons.lang.StringUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.security.core.userdetails.User;
@@ -73,7 +74,8 @@ public class AmbariLocalUserDetailsService implements UserDetailsService {
UserEntity user = userDAO.findLocalUserByName(username);
- if (user == null) {
+ if (user == null || !StringUtils.equals(user.getUserName(), username)) {
+ //TODO case insensitive name comparison is a temporary solution, until users API will change to use id as PK
log.info("user not found ");
throw new UsernameNotFoundException("Username " + username + " not found");
}
http://git-wip-us.apache.org/repos/asf/ambari/blob/2d18d2df/ambari-web/app/router.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/router.js b/ambari-web/app/router.js
index f19d621..fa147e2 100644
--- a/ambari-web/app/router.js
+++ b/ambari-web/app/router.js
@@ -271,7 +271,7 @@ App.Router = Em.Router.extend({
login: function () {
var controller = this.get('loginController');
- var loginName = controller.get('loginName').toLowerCase();
+ var loginName = controller.get('loginName');
controller.set('loginName', loginName);
var hash = misc.utf8ToB64(loginName + ":" + controller.get('password'));
var usr = '';
[2/2] ambari git commit: AMBARI-13997. Ambari force-writing usernames
to be lowercase in DB. (mpapirkovskyy)
Posted by mp...@apache.org.
AMBARI-13997. Ambari force-writing usernames to be lowercase in DB. (mpapirkovskyy)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/2fef316d
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/2fef316d
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/2fef316d
Branch: refs/heads/branch-2.1
Commit: 2fef316d52460217ab1c9d33272d0d83905708a9
Parents: 2e96a5d
Author: Myroslav Papirkovskyi <mp...@hortonworks.com>
Authored: Fri Nov 20 21:47:55 2015 +0200
Committer: Myroslav Papirkovskyi <mp...@hortonworks.com>
Committed: Fri Nov 20 21:48:21 2015 +0200
----------------------------------------------------------------------
.../org/apache/ambari/server/api/services/UserService.java | 1 +
.../java/org/apache/ambari/server/controller/AmbariServer.java | 1 +
.../main/java/org/apache/ambari/server/orm/dao/UserDAO.java | 6 +++---
.../security/authorization/AmbariLocalUserDetailsService.java | 4 +++-
ambari-web/app/router.js | 2 +-
5 files changed, 9 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/2fef316d/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java b/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
index 05ee6b1..76a6a08 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/api/services/UserService.java
@@ -31,6 +31,7 @@ import javax.ws.rs.core.UriInfo;
import org.apache.ambari.server.api.resources.ResourceInstance;
import org.apache.ambari.server.controller.spi.Resource;
+import org.apache.commons.lang.StringUtils;
import java.util.Collections;
http://git-wip-us.apache.org/repos/asf/ambari/blob/2fef316d/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java b/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
index 33bb95c..9d6e862 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
@@ -82,6 +82,7 @@ import org.apache.ambari.server.orm.entities.MetainfoEntity;
import org.apache.ambari.server.resources.ResourceManager;
import org.apache.ambari.server.resources.api.rest.GetResource;
import org.apache.ambari.server.scheduler.ExecutionScheduleManager;
+import org.apache.ambari.server.security.AmbariEntryPoint;
import org.apache.ambari.server.security.AmbariServerSecurityHeaderFilter;
import org.apache.ambari.server.security.CertificateManager;
import org.apache.ambari.server.security.SecurityFilter;
http://git-wip-us.apache.org/repos/asf/ambari/blob/2fef316d/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java b/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
index 12f975e..7f9d146 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
@@ -131,21 +131,21 @@ public class UserDAO {
@Transactional
public void create(Set<UserEntity> users) {
for (UserEntity user: users) {
- user.setUserName(user.getUserName().toLowerCase());
+// user.setUserName(user.getUserName().toLowerCase());
entityManagerProvider.get().persist(user);
}
}
@Transactional
public UserEntity merge(UserEntity user) {
- user.setUserName(user.getUserName().toLowerCase());
+// user.setUserName(user.getUserName().toLowerCase());
return entityManagerProvider.get().merge(user);
}
@Transactional
public void merge(Set<UserEntity> users) {
for (UserEntity user: users) {
- user.setUserName(user.getUserName().toLowerCase());
+// user.setUserName(user.getUserName().toLowerCase());
entityManagerProvider.get().merge(user);
}
}
http://git-wip-us.apache.org/repos/asf/ambari/blob/2fef316d/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java b/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
index 2aae8a0..5607cc5 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserDetailsService.java
@@ -27,6 +27,7 @@ import org.apache.ambari.server.orm.entities.MemberEntity;
import org.apache.ambari.server.orm.entities.PrincipalEntity;
import org.apache.ambari.server.orm.entities.PrivilegeEntity;
import org.apache.ambari.server.orm.entities.UserEntity;
+import org.apache.commons.lang.StringUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.security.core.userdetails.User;
@@ -73,7 +74,8 @@ public class AmbariLocalUserDetailsService implements UserDetailsService {
UserEntity user = userDAO.findLocalUserByName(username);
- if (user == null) {
+ if (user == null || !StringUtils.equals(user.getUserName(), username)) {
+ //TODO case insensitive name comparison is a temporary solution, until users API will change to use id as PK
log.info("user not found ");
throw new UsernameNotFoundException("Username " + username + " not found");
}
http://git-wip-us.apache.org/repos/asf/ambari/blob/2fef316d/ambari-web/app/router.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/router.js b/ambari-web/app/router.js
index 15b4c14..01a4b92 100644
--- a/ambari-web/app/router.js
+++ b/ambari-web/app/router.js
@@ -238,7 +238,7 @@ App.Router = Em.Router.extend({
login: function () {
var controller = this.get('loginController');
- var loginName = controller.get('loginName').toLowerCase();
+ var loginName = controller.get('loginName');
controller.set('loginName', loginName);
var hash = misc.utf8ToB64(loginName + ":" + controller.get('password'));
var usr = '';