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

Should use the options given in instantiation, but it doesnt #580

Closed
1 of 2 tasks
alesmenzelsocialbakers opened this issue May 14, 2019 · 0 comments · Fixed by #610
Closed
1 of 2 tasks

Should use the options given in instantiation, but it doesnt #580

alesmenzelsocialbakers opened this issue May 14, 2019 · 0 comments · Fixed by #610
Labels
enhancement New feature or request

Comments

@alesmenzelsocialbakers
Copy link

Note: for support questions, please use one of these channels: stackoverflow or slack

You want to:

  • report a bug
  • request a feature

Current behaviour

Creating new instance of engine.io and then attaching it to a http server does not carry on the path option given when instantiating. See repro:

Steps to reproduce (if the current behaviour is a bug)

const engine = require('engine.io');
const eio = engine({ path: '/eio' });
eio.attach(http.createServer());
// eio is listening on the default /engine.io path instead of /eio - one must give the options again in attach to make it work

Expected behaviour

Attaching the engine.io instance to the correct path specified in the constructor unless override is given in attach(srv, { path: '/override-eio' }).

Setup

  • engine.io version: latest

Other information (e.g. stacktraces, related issues, suggestions how to fix)

Options passed as parameter should respect constructor parameters.
https://github.com/socketio/engine.io/blob/master/lib/server.js#L438

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

Successfully merging a pull request may close this issue.

2 participants