Opened 9 years ago

Closed 9 years ago

#17119 closed Bug (fixed)

Cache Framework: wrong per-site cache behavior documentation

Reported by: Vanni Owned by: Dan Poirier
Component: Documentation Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


From documentation page: "The cache middleware caches every page that doesn't have GET or POST parameters".
AFAIK it is not true. URLs with GET parameters (like get (IMHO correctly) cached and POST parameters are simply ignored.

Attachments (1)

17119.diff (2.0 KB) - added by Calvin Spealman 9 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 9 years ago by vanni.totaro@…

comment:2 Changed 9 years ago by vanni.totaro@…

Type: UncategorizedBug

comment:3 Changed 9 years ago by vanni.totaro@…

Easy pickings: set

comment:4 Changed 9 years ago by Dan Poirier

Owner: changed from nobody to Dan Poirier
Triage Stage: UnreviewedAccepted

Looking at the cache code, doc is clearly wrong.

Changed 9 years ago by Calvin Spealman

Attachment: 17119.diff added

comment:6 Changed 9 years ago by Karen Tracey

Resolution: fixed
Status: newclosed

In [17090]:

Fix #17119: Update cache doc to match current implementation, which does (since r15705) cache pages with GET parameters. Thanks Vanni, poirier, and calvinspealman.

Note: See TracTickets for help on using tickets.
Back to Top