Skip to content

VI-5068 Drop the flow-poll cache-buster that 8.2 stores reject - #26

Merged
helgehoi merged 1 commit into
mainfrom
VI-5068-drop-flow-poll-cache-buster
Aug 26, 2026
Merged

helgehoi merged 1 commit into
mainfrom
VI-5068-drop-flow-poll-cache-buster

Conversation

@helgehoi

Copy link
Copy Markdown
Collaborator

Ticket

VI-5068 — [tamstool] Upgrade TAMS Store Browser to TAMS 8.2

The bug

Against an 8.2 store, /#/embed logs a 400 every 5 seconds:

GET https://…/flows/2928d8bb-…/?include_timerange=true&_t=1787786076960  400 (Bad Request)

Why

The growing-flow poll appends a cache-buster:

await api.get(`/flows/${flowId}?include_timerange=true&_t=${Date.now()}`);

added in 98ebed4 to bypass API Gateway and CDN caching. A TAMS 8.2 store validates query parameters and rejects the unknown _t with a 400, so every one of those polls has been failing against such a store — invisibly, since the loop ends in catch { /* skip */ }. It is part of why duration and LIVE never moved on a growing flow before #25.

The same request without the parameter is what the re-check path in #25 already issues, successfully, against the same store and the same flow — which is the direct evidence that _t is what the store objects to.

The parameter was not earning its place in the browser either: api.get already fetches with cache: "no-store".

The fix

Drop it. No replacement header — Cache-Control is not CORS-safelisted, and adding one would put the request at the mercy of each store's allowed-headers list for a problem no-store already covers.

Testing

🤖 Generated with Claude Code

The growing-flow poll appended `&_t=<now>` to bypass API Gateway and CDN
caching (98ebed4). A TAMS 8.2 store validates query parameters and answers an
unknown one with 400, so every one of those polls has been failing against
such a store -- invisibly, inside the catch that skips a failed source. The
same URL without the parameter is what the re-check path already uses
successfully.

api.get fetches with cache: "no-store", so the parameter was carrying no
weight in the browser either.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@helgehoi
helgehoi merged commit 43b6cea into main Aug 26, 2026
1 check passed
@helgehoi
helgehoi deleted the VI-5068-drop-flow-poll-cache-buster branch August 27, 2026 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant