diff options
| author | silverwind <me@silverwind.io> | 2020-05-22 03:45:34 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-05-21 21:45:34 -0400 |
| commit | 88fe7b5a720365e5c8f0a347730f895263465311 (patch) | |
| tree | 7d9de5b5c7d78878787dbe12c40ebe97791b2b52 /webpack.config.js | |
| parent | f6f49700cc0f5d66b0c1ff5dc3e310c777583e85 (diff) | |
| download | gitea-88fe7b5a720365e5c8f0a347730f895263465311.tar.gz gitea-88fe7b5a720365e5c8f0a347730f895263465311.zip | |
Move serviceworker to workbox and fix SSE interference (#11538)
* Move serviceworker to workbox and fix SSE interference
Instead of statically hardcoding every frontend asset, this uses a
type-based approach to cache all js,css and manifest.json requests.
This also fixes the issue that the service worker was interfering with
EventSource because it was unconditionally handling all requests which
this new implementation doesn't.
Fixes: https://github.com/go-gitea/gitea/issues/11092
Fixes: https://github.com/go-gitea/gitea/issues/7372
* rethrow error instead of logging
* await .register
* Revert "rethrow error instead of logging"
This reverts commit 043162ba1f18b98a4bf9635959fd28d16e839fc5.
* improve comment
* remove JSRenderer
* add version-based cache invalidation
* refactor
* more refactor
* remove comment
* rename item to fit cache name
Co-authored-by: guillep2k <18600385+guillep2k@users.noreply.github.com>
Diffstat (limited to 'webpack.config.js')
| -rw-r--r-- | webpack.config.js | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/webpack.config.js b/webpack.config.js index d6a632ad1f..b1038c407d 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -35,13 +35,20 @@ module.exports = { jquery: [ resolve(__dirname, 'web_src/js/jquery.js'), ], + serviceworker: [ + resolve(__dirname, 'web_src/js/serviceworker.js'), + ], icons: glob('node_modules/@primer/octicons/build/svg/**/*.svg'), ...themes, }, devtool: false, output: { path: resolve(__dirname, 'public'), - filename: 'js/[name].js', + filename: ({chunk}) => { + // serviceworker can only manage assets below it's script's directory so + // we have to put it in / instead of /js/ + return chunk.id === 'serviceworker' ? '[name].js' : 'js/[name].js'; + }, chunkFilename: 'js/[name].js', }, optimization: { |