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)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
111 Branch
Tracking Status
firefox110 --- wontfix
firefox111 --- fixed

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:

  1. inspect() fetches the server info. CDN caches the response with _expected=0 (eg. timetamp=42)
  2. a change is published on server, client syncs (eg. timestamp=99)
  3. CDN caches the latest the response with _expected=99
  4. inspect() fetches the server info, but obtains the cached response with _expected=0
  5. 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

No longer depends on: 1804277
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
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch

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 to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(mathieu)
Flags: needinfo?(mathieu)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: