Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
7 Oct 2016

Pull Request for Issue #271

Summary of Changes

303 is default 301 is expected here

Testing Instructions

$ curl -sD - -L "http://www.example.org/component/weblinks/weblink/50-movie-distributors-us/46->uncork-d-entertainment.html?Itemid=101&task=weblink.go" |head -n 3

HTTP/1.1 303 See other
Date: Tue, 04 Oct 2016 22:04:10 GMT
avatar zero-24 zero-24 - open - 7 Oct 2016
avatar yvesh
yvesh - comment - 7 Oct 2016

Looks fine for me, Thank you @zero-24

avatar zero-24
zero-24 - comment - 15 Oct 2016

Can someone take a look into the test problem or reboot travis? i can't see a issue that i produced

avatar 810
810 - comment - 15 Oct 2016

@zero-24 rebooted

avatar yvesh
yvesh - comment - 15 Oct 2016

@zero-24 rebooting, not always fixes it.. Did once already.. Travis crashs in one of three cases (and we do exactly three per PR).. Maybe it's time to set it to allow failure until we find a solution.

avatar 810
810 - comment - 15 Oct 2016

fixed, will merge it

avatar zero-24
zero-24 - comment - 15 Oct 2016

?

Add a Comment

Login with GitHub to post a comment