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

engine.io-client dependency broken, socket.io connections not working #715

Closed
stbuehler opened this issue Feb 1, 2018 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@stbuehler
Copy link

In a fresh install:

Right now the socket.io-client@^2.0.4 dependency depends on engine.io-client@~3.1.0, ending up as 3.1.4, and the socket.io@~2.0.4 pulls in engine.io@~3.1.0 (again ending up as 3.1.4).

But hackmd depends on engine.io-client@^1.8.2, ending up as 1.8.5 - which does not work with engine.io@3.1.4.

I think the engine.io-client@^1.8.2 dependency should be removed, and modulesDirectories should be set to only contain node_modules, not absolute paths (see webpack/webpack#472 (comment)); that way webpack should still find engine.io-client.

It seems webpack needs more memory and more time that way, but maybe you can figure out why (perhaps its including some modules more than once, and the dependencies should get fixed further to have only one version of those).

As a start increasing the engine.io-client dependency to engine.io-client@^3.1.0 should workaround the problem, but that is NOT a longterm solution!

@SISheogorath SISheogorath added the bug Something isn't working label Feb 1, 2018
@SISheogorath
Copy link
Contributor

Confirmed. I'll (try to) work on a solution this weekend.

@stbuehler
Copy link
Author

It seems path.resolve(__dirname, 'node_modules') is also needed; without that it complains about undefined eve (more might be missing, I think it stopped there).

But it works without path.resolve(__dirname, 'src') (I don't think that directory exists anyway).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants