﻿id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
5538	assertRedirects should assume http://testserver if it isn't provided	Russell Keith-Magee	nobody	"[6164] fixed #987, a problem with redirects allowing relative URIs. The fix is correct, but it makes the testing API slightly unwieldy; all calls to assertRedirects() now require that ``'http://testserver'`` be part of the asserted URL. 

The vast majority of redirection tests will be internal to the site, so requiring the full dummy host prefix only serves to muddy the tests. This is made worse by the fact that urlresolvers.reverse() doesn't include the testserver component, so assertRedirects(reverse('some_name')) doesn't work.

assertRedirects should be modified to inspect the test URL; if no '://' doesn't occur in the test url, 'http://testserver' should be prepended for test purposes. i.e.,

``assertRedirects('/some/url')`` is equivalent to ``assertRedirects('http://testserver/some/url')``
``assertRedirects('http://otherserver/some/url')`` would be untouched
"		new	Testing framework	dev			assertRedirects testserver		Unreviewed	0	0	0	0	0	0
