Closed
Bug 741155
Opened 14 years ago
Closed 12 years ago
[bedrock] Investigate if Vary: Cookie is necessary
Categories
(www.mozilla.org :: Bedrock, defect)
www.mozilla.org
Bedrock
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wenzel, Unassigned)
Details
Bedrock varies by cookie (and WebTrends etc set cookies). This probably leads to a lot of duplicates in the load balancer.
Is this necessary? AMO doesn't do this, for example.
If we don't intentionally set this, it might be set by SessionMiddleware. AMO has a NoVarySessionMiddleware, with explanation:
<https://github.com/mozilla/zamboni/blob/8719e16fb5c386a6d1ca301975f965f228ee435f/apps/amo/middleware.py#L82>
|
Reporter | |
Updated•14 years ago
|
Assignee: nobody → jlong
|
|
Reporter | |
Comment 1•14 years ago
|
||
https://github.com/mozilla/bedrock/commit/6861f343910d2e06a907836cddb19e6f15f2dd96
Please check which middlewares we might be able to remove altogether.
Severity: blocker → critical
|
||
Comment 2•14 years ago
|
||
This has been pushed to mozilla/master and awaits deployment.
Severity: critical → blocker
|
|
Reporter | |
Comment 3•14 years ago
|
||
All right. Keeping this open for now because we might be able to remove this middleware altogether.
Severity: blocker → major
|
Assignee | |
Updated•14 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
|
|
Assignee | |
Updated•13 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
|
||
Updated•12 years ago
|
Component: General → Bedrock
|
|
||
Updated•12 years ago
|
Assignee: jlong → nobody
|
|
||
Comment 4•12 years ago
|
||
Can I close this now?
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
|
Reporter | |
Comment 5•12 years ago
|
||
Sure, since it was fixed ;)
You need to log in
before you can comment on or make changes to this bug.
Description
•