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
15499	patch_cache_control should replace public/private setting if already set	Andreas Sommer	Andreas Sommer	"In the following scenario

{{{
@_cache_control_iff_user_is_anonymous(public = True, must_revalidate = False, max_age = 60)
@cache_control(private = True, must_revalidate = True, max_age = 1200)
@vary_on_cookie
def myView(request):
    ...
}}}

I am applying the private caching attribute by default, and want to replace ""private"" with ""public"" iff the user is not logged in (for more aggressive caching).

But what I get with the above code is ""Cache-Control: ''public, private'', must-revalidate=False, max-age=60"". So instead of just adding the ""public"" attribute, one should as well remove the ""private"" attribute.

See attached diff for a simple solution.
"	Bug	closed	HTTP handling	dev	Normal	fixed			Ready for checkin	1	0	0	0	0	0