Remove split-script configuration property
split-script feature isn't documented anywhere, and it doesn't seem to be entirely correct anyway?
This feature is only active when both
split-script are enabled.
What this feature appears to do¶
When a normal (not WidgetSet, nor plain HTML) session bootstraps, it follows this pattern:
- A page (based on
Boot.jsis loaded (
split-script is enabled, the second step is split up into:
- Upgrading the progressive bootstrap page (the rest)
This seems to be some attempt at caching part of the loaded script since for the first part (the skeletons) the
Cache-Control header is added:
Problems with this feature¶
When the feature is enabled it doesn't appear to break anything, but the cache is always defeated by the fact that the request contains the session id and script id as parameters, which are randomly generated for every session.
Since I fail to see the benefit, and the feature was never actually advertised anyway (it was never mentioned in release notes or any documentation), I propose we purge it from Wt's source code altogether, which nicely simplifies
Updated by Korneel Dumon 6 months ago
- Status changed from Review to Implemented @Emweb
- Assignee changed from Korneel Dumon to Roel Standaert