Skip to content

Optimize Emscripten REST long-poll requests - #410

Merged
drslebedev merged 2 commits into
mainfrom
fix_firefox
Sep 3, 2026
Merged

Optimize Emscripten REST long-poll requests#410
drslebedev merged 2 commits into
mainfrom
fix_firefox

Conversation

@drslebedev

Copy link
Copy Markdown
Contributor

This PR reduces unnecessary browser-side work for Emscripten REST requests:

  • Omits Content-Type from GET and long-poll requests, avoiding unnecessary CORS OPTIONS preflights. POST requests still include it.
  • Uses EMSCRIPTEN_FETCH_REPLACE to bypass Emscripten’s implicit IndexedDB lookup, since REST responses are not persisted.

This is part of the browser performance optimization work for dashboards with many subscriptions, particularly for Firefox.

Send Content-Type only for POST requests while retaining Accept for GET
and long-poll requests. This avoids unnecessary OPTIONS requests without
changing response content negotiation.

Signed-off-by: drslebedev <dr.s.lebedev@gmail.com>
Use EMSCRIPTEN_FETCH_REPLACE to skip Emscripten's implicit IndexedDB
lookup. REST responses are not persisted and should be fetched directly.

Signed-off-by: drslebedev <dr.s.lebedev@gmail.com>
@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

@drslebedev
drslebedev merged commit bf1a8df into main Sep 3, 2026
11 checks passed
@drslebedev
drslebedev deleted the fix_firefox branch September 3, 2026 11:40
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.

2 participants