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

webpack server: add disableHostCheck option #915

Merged
merged 2 commits into from
Oct 5, 2017
Merged

Conversation

edemaine
Copy link
Member

@edemaine edemaine commented Oct 2, 2017

Allow server to be accessed from anywhere, which is useful for testing (see #911). This potentially reveals the source code to the world, but this should not be a concern for testing open-source software.

Allow server to be accessed from anywhere, which is useful for
testing.  This potentially reveals the source code to the world,
but this should not be a concern for testing open-source software.
Copy link
Member

@ry-randall ry-randall left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@ry-randall
Copy link
Member

@edemaine @kevinbarabash Would you like me to go ahead and merge this?

@edemaine edemaine merged commit c4fed02 into KaTeX:master Oct 5, 2017
@edemaine
Copy link
Member Author

edemaine commented Oct 5, 2017

Thanks @rrandallcainc for the instant review. Somehow, last I checked, I still couldn't merge, but no problem now. Thanks for the reminder!

@edemaine
Copy link
Member Author

@ylemkimon I believe the recent webpack revamp PRs have disabled this functionality. Could we add it back?

@ylemkimon
Copy link
Member

ylemkimon commented Jan 31, 2018

@edemaine We still disableHostCheck = true, however it seems that webpack-dev-server CLI has localhost as default value for host. I'll send a PR setting the host to 0.0.0.0, and meanwhile you can do npm start -- --host 0.0.0.0.

@edemaine
Copy link
Member Author

edemaine commented Feb 1, 2018

#1122 fixed this. Thanks @ylemkimon!

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.

3 participants