You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by "Vitaly Brodetskyi (JIRA)" <ji...@apache.org> on 2014/02/19 16:22:20 UTC

[jira] [Updated] (AMBARI-4738) Add unit test that verifies escaped urls(ProxyService)

     [ https://issues.apache.org/jira/browse/AMBARI-4738?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vitaly Brodetskyi updated AMBARI-4738:
--------------------------------------

    Attachment: AMBARI-4738.patch

> Add unit test that verifies escaped urls(ProxyService)
> ------------------------------------------------------
>
>                 Key: AMBARI-4738
>                 URL: https://issues.apache.org/jira/browse/AMBARI-4738
>             Project: Ambari
>          Issue Type: Bug
>          Components: test
>            Reporter: Vitaly Brodetskyi
>            Assignee: Vitaly Brodetskyi
>         Attachments: AMBARI-4738.patch
>
>
> 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.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)