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
6218	Require trailing slash always on MEDIA_URL	Michael Toomim <toomim@…>	Chris Heisel	"This is a follow-up patch to the resolution of ticket 6198.

Right now, MEDIA_URL requires a trailing slash if it contains a suffix beyond the domain name, and otherwise the slash is optional. This is an artifact of using urljoin() to combine MEDIA_URL with the url suffix. (Urljoin() was implements relative urls in RFC1808, slightly different than gluing paths together.)

The only real problem is that if you forget the trailing slash from MEDIA_URL, it fails in an odd way, by deleting the path after the last trailing slash. For instance, ""http://localhost/dir"" combined with ""image1.png"" will become ""http://localhost/image1.png"".

This patch replaces the use of ""urljoin(MEDIA_URL, suffix)"" with ""MEDIA_URL + suffix"" and updates the documentation. Now it would fail as ""http://localhost/dirimage1.png"" which is easier to debug."		closed	Core (Other)	dev		fixed	easy-pickings		Accepted	1	0	0	1	0	0