Prevent minification of .html files
#17349
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ebpack-plugin This described in #17330 (comment), this should prevent the newly introduced version of `html-webpack-plugin` from minifying `.html` files, like e.g. `index.html`, `jitsi.html`, et cetera … Quoting @jryans via: #17330 (comment) > The content of the `index.html` file is not a supported API surface, so it might change at any time. > > This document minification was not done on purpose, but instead it happened as a side effect of upgrading. We would happily accept a PR to fix this, but it is also not a priority for the core team. Could please someone test and, hopefully, accept this change back to not minifying the `.html` files?
jryans
changed the title
.html files generated by new verison html-webpack-plugin.html files
jryans
approved these changes
May 18, 2021
|
Thank you very much, @jryans! PS: I guess my branch can be deleted, right? |
|
sure, it's a branch on your repo which means it's your choice :) |
netbsd-srcmastr
pushed a commit
to NetBSD/pkgsrc
that referenced
this pull request
Jun 6, 2021
Changes in [1.7.29](https://github.com/vector-im/element-web/releases/tag/v1.7.29) (2021-05-24) =============================================================================================== [Full Changelog](element-hq/element-web@v1.7.29-rc.1...v1.7.29) ## Security notice Element Web 1.7.29 fixes (by upgrading to olm 3.2.3) an issue in code used for decrypting server-side stored secrets. The issue could potentially allow a malicious homeserver to cause a stack buffer overflow in the affected function and to control that function's local variables. ## All changes * Upgrade to React SDK 3.22.0 and JS SDK 11.1.0 * [Release] Bump libolm dependency, and update package name [\#17456](element-hq/element-web#17456) Changes in [1.7.29-rc.1](https://github.com/vector-im/element-web/releases/tag/v1.7.29-rc.1) (2021-05-19) ========================================================================================================= [Full Changelog](element-hq/element-web@v1.7.28...v1.7.29-rc.1) * Upgrade to React SDK 3.22.0-rc.1 and JS SDK 11.1.0-rc.1 * Translations update from Weblate [\#17384](element-hq/element-web#17384) * Prevent minification of `.html` files [\#17349](element-hq/element-web#17349) * Update matrix-widget-api/react-sdk dependency reference [\#17346](element-hq/element-web#17346) * Add `yarn start:https` [\#16989](element-hq/element-web#16989) * Translations update from Weblate [\#17239](element-hq/element-web#17239) * Remove "in development" flag from voice messages labs documentation [\#17204](element-hq/element-web#17204) * Add required webpack+jest config to load Safari support modules [\#17193](element-hq/element-web#17193) Changes in [1.7.28](https://github.com/vector-im/element-web/releases/tag/v1.7.28) (2021-05-17) =============================================================================================== [Full Changelog](element-hq/element-web@v1.7.28-rc.1...v1.7.28) ## Security notice Element Web 1.7.28 fixes (by upgrading to matrix-react-sdk 3.21.0) a low severity issue (GHSA-8796-gc9j-63rv) related to file upload. When uploading a file, the local file preview can lead to execution of scripts embedded in the uploaded file, but only after several user interactions to open the preview in a separate tab. This only impacts the local user while in the process of uploading. It cannot be exploited remotely or by other users. Thanks to [Muhammad Zaid Ghifari](https://github.com/MR-ZHEEV) for responsibly disclosing this via Matrix's Security Disclosure Policy. ## All changes * Upgrade to React SDK 3.21.0 and JS SDK 11.0.0 Changes in [1.7.28-rc.1](https://github.com/vector-im/element-web/releases/tag/v1.7.28-rc.1) (2021-05-11) ========================================================================================================= [Full Changelog](element-hq/element-web@v1.7.27...v1.7.28-rc.1) * Upgrade to React SDK 3.21.0-rc.1 and JS SDK 11.0.0-rc.1 * Switch back to release version of `sanitize-html` [\#17231](element-hq/element-web#17231) * Bump url-parse from 1.4.7 to 1.5.1 [\#17199](element-hq/element-web#17199) * Bump lodash from 4.17.20 to 4.17.21 [\#17205](element-hq/element-web#17205) * Bump hosted-git-info from 2.8.8 to 2.8.9 [\#17219](element-hq/element-web#17219) * Disable host checking on the webpack dev server [\#17194](element-hq/element-web#17194) * Bump ua-parser-js from 0.7.23 to 0.7.24 [\#17190](element-hq/element-web#17190) Changes in [1.7.27](https://github.com/vector-im/element-web/releases/tag/v1.7.27) (2021-05-10) =============================================================================================== [Full Changelog](element-hq/element-web@v1.7.27-rc.1...v1.7.27) * Upgrade to React SDK 3.20.0 and JS SDK 10.1.0 Changes in [1.7.27-rc.1](https://github.com/vector-im/element-web/releases/tag/v1.7.27-rc.1) (2021-05-04) ========================================================================================================= [Full Changelog](element-hq/element-web@v1.7.26...v1.7.27-rc.1) * Upgrade to React SDK 3.20.0-rc.1 and JS SDK 10.1.0-rc.1 * Translations update from Weblate [\#17160](element-hq/element-web#17160) * Document option for obeying asserted identity [\#17008](element-hq/element-web#17008) * Implement IPC call to Electron to set language [\#17052](element-hq/element-web#17052) * Convert Vector skin react components to Typescript [\#17061](element-hq/element-web#17061) * Add code quality review policy [\#16980](element-hq/element-web#16980) * Register RecorderWorklet from react-sdk [\#17013](element-hq/element-web#17013) * Preload Inter font to avoid FOIT on slow connections [\#17039](element-hq/element-web#17039) * Disable `postcss-calc`'s noisy `warnWhenCannotResolve` option [\#17041](element-hq/element-web#17041)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As described in #17330 (comment), this should prevent the newly introduced version of
html-webpack-pluginfrom minifying.htmlfiles, like e.g.index.html,jitsi.html, et cetera …Quoting @jryans via #17330 (comment):
Could someone with a dev environment please test and, hopefully, accept this change back to not minifying the
.htmlfiles?Thank you very much.
—
Signed-off-by: fooness