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.

Sign up for GitHub

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

Jump to bottom

127.0.0.1 override every host set in constructor #568

Closed
Mokto opened this issue Jan 3, 2018 · 3 comments
Closed

127.0.0.1 override every host set in constructor #568

Mokto opened this issue Jan 3, 2018 · 3 comments

Comments

@Mokto
Copy link

Mokto commented Jan 3, 2018

Hi,

With ioredis > 3.1.2, it is not possible to connect to adresses other than 127.0.0.1. The host is automatically replaced by 127.0.0.1.

I tried to use new Redis(object), new Redis(6379, host). Both are being replaced.

Thanks.
@VineyShih
Copy link

I am facing the similar issue and solving it by #365.
Duplicate of #365

dreambo8563 added a commit to dreambo8563/ts_node that referenced this issue Jan 23, 2018
@dreambo8563
fix(<redis session>): fixed the issue which not store session into redis
c7494d4 
* user session 没有存入redis
* 因为有issue,需要提前创建好redisclient,否则host 会被覆盖成127.0.0.1
https://stackoverflow.com/search?q=docker+redis+express+session
redis/ioredis#568
@stale
Copy link

stale bot commented Feb 18, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 7 days if no further activity occurs, but feel free to re-open a closed issue if needed.

@stale stale bot added the wontfix label Feb 18, 2018
@stale stale bot closed this as completed Feb 25, 2018
@divmgl
Copy link

divmgl commented Mar 14, 2018

Can we get a response on this? I'm just now facing this issue as well. Is the solution to downgrade?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Mokto @divmgl @VineyShih