vishalsuvagia opened a new pull request, #3938: URL: https://github.com/apache/ambari/pull/3938
Team, apologies for pushing the commit directly without having a PR, it was accidentally done due to a missed check for push to a right branch instead got pushed to the tracking branch. Will take care about this in future contributions. ## What changes were proposed in this pull request? Issue: User authentication fails for previously sync'd LDAP / AD based users. Cause: the authentication-key was being split before checking if the authentication type required is local or remote based, hence it fails with a NullPointerException. (Please fill in changes proposed in this fix) Changes made: Updated the implementation to add a check for auth type before adding split for authentication-key ## How was this patch tested? validated changes on a local cluster with AD setup (Please explain how this patch was tested. Ex: unit tests, manual tests) (If this patch involves UI changes, please attach a screen-shot; otherwise, remove this) Please review [Ambari Contributing Guide](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) before opening a pull request. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@ambari.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@ambari.apache.org For additional commands, e-mail: dev-h...@ambari.apache.org
