-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18271/
-----------------------------------------------------------
Review request for Ambari, Andrew Onischuk and Nate Cole.
Bugs: AMBARI-4738
https://issues.apache.org/jira/browse/AMBARI-4738
Repository: ambari
Description
-------
For completeness, we should have a unit test that verifies escaped urls are
covered. IE:
/proxy?url=http%3a%2f%2fserver%3a8188%2fws%2fv1%2fapptimeline%2fHIVE_QUERY_ID%3ffields=events%2cprimaryfilters%26limit=10%26primaryFilter=user%3ahiveuser1
In fact, requiring that syntax gets out of the substring/replace business and
just use the url parameter and make a URI out of it.
Diffs
-----
ambari-server/src/test/java/org/apache/ambari/server/proxy/ProxyServiceTest.java
b90af09
Diff: https://reviews.apache.org/r/18271/diff/
Testing
-------
-------------------------------------------------------
T E S T S
-------------------------------------------------------
Listening for transport dt_socket at address: 5007
Running org.apache.ambari.server.proxy.ProxyServiceTest
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 68.465 sec
Results :
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0
Thanks,
Vitalyi Brodetskyi
