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.

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

fix(config): add ability to serve generated static files for ssr dev #59

Merged
merged 2 commits into from
Sep 8, 2017

Conversation

mengtzu
Copy link
Contributor

@mengtzu mengtzu commented Sep 6, 2017

Commit Message For Review

Adds serving for static files in the root directory for server side rendering.

REASON FOR CHANGE:

Without this, when using SSR in dev requests for main.js, styles.css and their friends will simply return the output of the render callback. In most apps this will be the rendered HTML intended for a 404.

Copy link
Contributor

@mattcompiles mattcompiles left a comment

Choose a reason for hiding this comment

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

Could this cause any prod issues where you could access other files through the server? Should this only be active on dev maybe?

@mengtzu
Copy link
Contributor Author

mengtzu commented Sep 7, 2017

@MattsJones sure, we're assuming outside dev the files are probably on an s3 bucket somewhere or otherwise not served by our express server, as I understand it.

@mengtzu
Copy link
Contributor Author

mengtzu commented Sep 7, 2017

@MattsJones added dev check

Copy link
Contributor

@mattcompiles mattcompiles left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants