You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@ambari.apache.org by Oliver Szabo <os...@hortonworks.com> on 2016/11/29 12:20:04 UTC

Review Request 54159: Log Search external login credential let every user in

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54159/
-----------------------------------------------------------

Review request for Ambari, Miklos Gergely and Robert Nettleton.


Bugs: AMBARI-19010
    https://issues.apache.org/jira/browse/AMBARI-19010


Repository: ambari


Description
-------

Jersey client does not throw exception during invalid login request, so we need to check the status code


Diffs
-----

  ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/ExternalServerClient.java 1207373 

Diff: https://reviews.apache.org/r/54159/diff/


Testing
-------

tested manually. (failed with invalid credential, successful with valid credential)


Thanks,

Oliver Szabo