Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Gzip support for JSON pages #571

Merged
merged 18 commits into from
Dec 31, 2016
Merged

Add Gzip support for JSON pages #571

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
bc2774a
Server JSON pages directly from the filesystem.
arunoda Dec 30, 2016
b6daa28
Make Json pages even if there's an error.
arunoda Dec 30, 2016
6dd5db3
Implement much better page serving.
arunoda Dec 30, 2016
bff2435
Use JsonPagesPlugin in the production mode as well.
arunoda Dec 30, 2016
901915d
Add gzip support for JSON pages.
arunoda Dec 30, 2016
6401e33
Merge master.
arunoda Dec 30, 2016
3af72f1
Use glob-promise instead of recursive-readdir
arunoda Dec 30, 2016
81ba173
Handle renderStatic 404 properly.
arunoda Dec 30, 2016
298629f
Simply the gzip code.
arunoda Dec 30, 2016
b0b04a7
Cache already read JSON pages.
arunoda Dec 30, 2016
1e0e7a8
Change JSON pages extension to .json.
arunoda Dec 30, 2016
e24f261
Fix HMR related issue.
arunoda Dec 30, 2016
7a04f38
Fix hot-reload for .json solely on server.
arunoda Dec 31, 2016
79d566c
Properly clear cache on hot-reloader.
arunoda Dec 31, 2016
352fe77
Convert .js pages into .json page right inside the plugin.
arunoda Dec 31, 2016
20d01ba
Fix gzipping .json pages.
arunoda Dec 31, 2016
343cb92
Remove unwanted json pages cleanup.
arunoda Dec 31, 2016
370189b
Get rid of deprecated fs.exists for fs.access
arunoda Dec 31, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove unwanted json pages cleanup.
  • Loading branch information
@arunoda
arunoda committed Dec 31, 2016
commit 343cb924745f82d877eadac14ea054ef2be079b6
8 changes: 0 additions & 8 deletions server/build/plugins/json-pages-plugin.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,6 @@ export default class JsonPagesPlugin {
.keys(compilation.assets)
.filter((filename) => /^bundles\/pages.*\.js$/.test(filename))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This regex doesn't match the way one could wish for in windows, causing pages not to be compiled to json, causing issue #597 thus making most tests fail on windows.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @msand for figuring this out.


const jsonPages = Object
.keys(compilation.assets)
.filter((filename) => /^bundles\/pages.*\.json$/.test(filename))

// Delete existing json pages
// Otherwise, we might keep JSON pages for deleted pages.
jsonPages.forEach((pagePath) => delete compilation.assets[pagePath])

pages.forEach((pageName) => {
const page = compilation.assets[pageName]
delete compilation.assets[pageName]
Expand Down