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

Fix: bind Redis to localhost #647

Merged
merged 1 commit into from Nov 11, 2015
Merged

Fix: bind Redis to localhost #647

merged 1 commit into from Nov 11, 2015

Conversation

runa
Copy link
Contributor

@runa runa commented Nov 11, 2015

Having it bound to the public addresses is a security problem.
See http://antirez.com/news/96

Having it bound to the public addresses is a security problem.
See http://antirez.com/news/96
@arikfr arikfr changed the title bind redis to localhost Fix: bind Redis to localhost Nov 11, 2015
arikfr added a commit that referenced this pull request Nov 11, 2015
Fix: bind Redis to localhost
@arikfr arikfr merged commit 91806ed into getredash:master Nov 11, 2015
@arikfr
Copy link
Member

arikfr commented Nov 11, 2015

Thanks!

@matangover
Copy link

I just found out my re:dash server was compromised multiple times last month because it used the previous (default) configuration which didn't include the 'bind' directive -- apparently some botnets (specifically XorDDOS) are going around taking advantage of this redis 'feature'. See e.g. here. I was not aware that leaving the redis port open to the world creates a security hole. With the fixed configuration by @runa (thanks!), it wouldn't be a problem for new installations, even if the redis port is left open.

Is there any way to alert current users of older re:dash versions (before v0.8.2.b1181) about this?

@arikfr
Copy link
Member

arikfr commented Dec 14, 2015

Unfortunately there is no way to really reach everyone, but I will mention this on the mailing list with the next update.

@matangover
Copy link

Great. Thanks.

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.

None yet

3 participants