github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-23 02:19:27 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

DEV: Don't use ?. in bootstrap-json (#15162)

Browse Source 
That code is not transpiled, so it doesn't work on older node versions.
This commit is contained in:
Jarek Radosz 2021-12-01 22:04:56 +01:00 committed by GitHub
parent bd140948e3
commit 1b3d124a4e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
app/assets/javascripts/discourse/lib/bootstrap-json/index.js
View File

@ -171,8 +171,13 @@ function replaceIn(bootstrap, template, id, headers, baseURL) {
function extractPreloadJson(html) {
const dom = new JSDOM(html);
return dom.window.document.querySelector("#data-preloaded")?.dataset
?.preloaded;
const dataElement = dom.window.document.querySelector("#data-preloaded");
if (!dataElement || !dataElement.dataset) {
return;
}
return dataElement.dataset.preloaded;
}
async function applyBootstrap(bootstrap, template, response, baseURL, preload) {
@ -264,17 +269,16 @@ async function handleRequest(proxy, baseURL, req, res) {
res.set("content-security-policy", newCSP);
}
const isHTML = response.headers.get("content-type")?.startsWith("text/html");
const contentType = response.headers.get("content-type");
const responseText = await response.text();
const preload = isHTML ? extractPreloadJson(responseText) : null;
if (preload) {
if (contentType && contentType.startsWith("text/html")) {
const html = await buildFromBootstrap(
proxy,
baseURL,
req,
response,
preload
extractPreloadJson(responseText)
);
res.set("content-type", "text/html");
res.send(html);