[
https://issues.apache.org/jira/browse/MESOS-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14034071#comment-14034071
]
Benjamin Hindman commented on MESOS-1448:
-----------------------------------------
Since the changes are in stout it's a bit tougher to test. One suggestion is to
add a test using libprocess's TemporaryRedirect: create a mock libprocess
process with two endpoints, one which returns a
process::http::TemporaryRedirect to the other endpoint, then put an EXPECT_CALL
on the other endpoint and have it return a process::http::OK with some
arbitrary string that you can check for. The weirdness is having to put this
test in libprocess (since it uses libprocess) but I think we can make a case
for testing redirect functionality (i.e., that returning a TemporaryRedirect is
properly formatted).
On the flip side, what did you mean by "break the code out so we can verify the
option was set without actually making a HTTP request"?
> Mesos Fetcher doesn't support URLs that have 30X redireects
> -----------------------------------------------------------
>
> Key: MESOS-1448
> URL: https://issues.apache.org/jira/browse/MESOS-1448
> Project: Mesos
> Issue Type: Bug
> Components: slave
> Reporter: Tom Arnfeld
> Assignee: Tom Arnfeld
>
> The mesos-fetcher program doesn't follow 30X redirects for http:// URLs.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
