1142896 - add Pragma: no-cache,Cache-Control:no-cache in the request header of double 302 redirect

Status: RESOLVED INVALID

(Core :: Networking: Cache, defect)

Description

[disound]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.76 Safari/537.36

Steps to reproduce:

User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Firefox/38.0

if there is a double redirect, means request A redirect to B, and B redirect 
to C, FF will add Pragma: no-cache,Cache-Control:no-cache in the request 
header of C which lead to css/js files not cached. 
But if A redirect to C or B redirect to C don't have this issue. 
Not exist in other browsers like IE/Chrome. 


Actual results:

css/js files not cached which lead to great performance issue

Pragma: no-cache
Cache-Control: no-cache
added in the request header


Expected results:

don't add Pragma: no-cache,Cache-Control:no-cache in the request header

Comment 1

[:Gijs (he/him)]

Honza, thoughts?

Comment 2

[disound]

any updates?

Comment 3

[Honza Bambas (:mayhemer)]

This is intentional.  Since we set either LOAD_BYPASS_CACHE or VALIDATE_ALWAYS load flag on the channel, we want any intermediate caches revalidate their content as well.

See http://hg.mozilla.org/mozilla-central/annotate/bc85c479668a/netwerk/protocol/http/nsHttpChannel.cpp#l738

Comment 4

[disound]

Is it the same issue?
My issue is double 302 redirect will add Pragma: no-cache,Cache-Control:no-cache in the request header which lead to not cache css/js files.
The cpp file above seems fix for proxy cache problem.

Comment 5

[Honza Bambas (:mayhemer)]

(In reply to disound from comment #4)
> Is it the same issue?
> My issue is double 302 redirect will add Pragma:
> no-cache,Cache-Control:no-cache in the request header which lead to not
> cache css/js files.
> The cpp file above seems fix for proxy cache problem.

Sorry, I don't follow what leads you to reopening of this bug.  Please state it clearly.

Comment 6

[Honza Bambas (:mayhemer)]

Closing on lack of information.