[GitHub] cloudstack pull request #2009: CLOUDSTACK-9369 Fixed Ldap regression

2017-03-20 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/2009#discussion_r106854397 --- Diff: server/src/com/cloud/api/auth/DefaultLoginAPIAuthenticatorCmd.java --- @@ -166,7 +166,7 @@ public String authenticate(String command, Map par

[GitHub] cloudstack pull request #2009: CLOUDSTACK-9369 Fixed Ldap regression

2017-03-20 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/2009#discussion_r106849093 --- Diff: server/src/com/cloud/api/auth/DefaultLoginAPIAuthenticatorCmd.java --- @@ -166,7 +166,7 @@ public String authenticate(String command, Map params

[GitHub] cloudstack pull request #2009: CLOUDSTACK-9369 Fixed Ldap regression

2017-03-17 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/2009 CLOUDSTACK-9369 Fixed Ldap regression Ldap auto creation of accounts is broken due to the security fix for CLOUDSTACK-9369. There was an explicit check to not allow login incase the