[jira] [Commented] (MESOS-1448) Mesos Fetcher doesn't support URLs that have 30X redireects

["Tom Arnfeld (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MESOS-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14033596#comment-14033596 ] 

Tom Arnfeld commented on MESOS-1448:
------------------------------------

I've submitted a patch to the review board that simply includes the {{CURLOPT_FOLLOWLOCATION}} option in stout.

I'm not sure I should just be making changes to stout in the mesos code base, since it's a 3rdparty library? Should this change go somewhere else, too?

https://reviews.apache.org/r/22675/

[~ijimenez] should I assign myself to this, or have you started work as well?

On the subject of unit tests, i'd love to add some test coverage for this however I'm in two minds.

 - Should we rely on an external resource that sends a Location header?
 - Should we spawn a little local HTTP server (overkill?) that behaves like we want?
 - Break the code out so we can verify the option was set without actually making a HTTP request

Any other suggestions?

> 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: Isabel Jimenez
>
> The mesos-fetcher program doesn't follow 30X redirects for http:// URLs.



--
This message was sent by Atlassian JIRA
(v6.2#6252)