Skip to content

Proxy exact paths to express #19

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Stieneee
Copy link
Contributor

Split from #17

@mathieutu
Copy link
Owner

Could you tell me what is your browser please? Did you try on another one?

I know that there are some recent issues with the webpack proxy, and Firefox.

@mathieutu
Copy link
Owner

mathieutu commented Jul 12, 2019

For me it's the same that facebook/create-react-app#6720, due to https://bugzilla.mozilla.org/show_bug.cgi?id=1536843.

In all cases, the proxy service and the HMR one are not part of this package, which is simply passing theses parameters in vue-cli services, so we should not merge theses specific modifications in here 😕.

Can you disable the proxy in the package, and try to enable it directly by following instructions here: https://cli.vuejs.org/config/#devserver-proxy and tell me if it's different?

@Stieneee
Copy link
Contributor Author

I am having a difficult time understanding how this is related to #6720 above. The issue exists due the fact that the original proxy configuration was proxying all requests from devServer to the express server.

I followed the instructions at https://cli.vuejs.org/config/#devserver-proxy to create the proxyConfig object to selectively proxy only the requests that should be sent to express.

Looking back #17 the first image shows the devServer proxy error /sockjs-node this is because the express server is not serving a /sockjs-node route.

Original
Frontend --> devServer -(proxy: *)-> express
With PR
Frontend --> devServer -(proxy: /socket.io, proxy: /bar, ...)-> express

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.

2 participants