Closed
Bug 1809884
Opened 1 year ago
Closed 1 year ago
inspect() does not bust cache when fetching the latest changes from server
Categories
(Firefox :: Remote Settings Client, defect)
Tracking
()
RESOLVED
FIXED
111 Branch
People
(Reporter: leplatrem, Assigned: leplatrem)
Details
Attachments
(1 file)
inspect()
pull the server information, but does not bust cache (it uses ?_expected=0
).
When using the devtools, the UI compares the server timestamp with the local timestamp.
With the following timeline, the timestamps will be different:
inspect()
fetches the server info. CDN caches the response with_expected=0
(eg. timetamp=42)- a change is published on server, client syncs (eg. timestamp=99)
- CDN caches the latest the response with
_expected=99
inspect()
fetches the server info, but obtains the cached response with_expected=0
- the two obtained timestamps are different whereas the client is up-to-date
We should bust the cache in inspect()
.
Since it's only used by DevTools (directly or via clearAll()
), we don't risk anything with regards to the server traffic
Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Assignee: nobody → mathieu
Status: NEW → ASSIGNED
Pushed by mleplatre@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b63a6959d471 Bust cache when fetching changes in inspect() r=gbeckley
Comment 3•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox111:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
Comment 4•1 year ago
|
||
The patch landed in nightly and beta is affected.
:leplatrem, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox110
towontfix
.
For more information, please visit auto_nag documentation.
Flags: needinfo?(mathieu)
Assignee | ||
Updated•1 year ago
|
Flags: needinfo?(mathieu)
You need to log in
before you can comment on or make changes to this bug.
Description
•