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

Updated invalid devServer.proxy example code #2320

Closed
wants to merge 1 commit into from
Closed

Updated invalid devServer.proxy example code #2320

wants to merge 1 commit into from

Conversation

petterm
Copy link

@petterm petterm commented Jun 29, 2018

It seems the last proxy example omitted the array format used in the example before it. Initial example did not work with webpack 4.13.0 and webpack-dev-server 3.1.4. The updated version worked fine.

It seems the last proxy example had an invalid config format. Tested with webpack 4.13.0 and webpack-dev-server 3.1.4.
Copy link
Member

@EugeneHlushko EugeneHlushko left a comment

Choose a reason for hiding this comment

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

By looking at proxy schema i can tell that it looks like a bug that you found and need to file in webpack-dev-server repository, if an object isnt supported anymore they must change the validation.

@petterm
Copy link
Author

petterm commented Jul 5, 2018

Posted bug report for webpack-dev-server at webpack/webpack-dev-server#1438

@EugeneHlushko
Copy link
Member

Thank you @petterm lets wait and hear from them to decide on what steps to take with this PR (close or merge)

@petterm
Copy link
Author

petterm commented Sep 5, 2018

As issue webpack/webpack-dev-server#1438 has been accepted as a bug report this pull request is no longer relevant.

@petterm petterm closed this Sep 5, 2018
@aoberoi
Copy link

aoberoi commented Sep 26, 2018

i just wanted to say THANK YOU for filing the bug report and creating this PR - i've spent over an hour trying to figure out why the example in the docs wasn't working as expected.

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

Successfully merging this pull request may close these issues.

4 participants