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

redis uri is not parsed correctly #41

Closed
yads opened this issue Apr 10, 2015 · 1 comment
Closed

redis uri is not parsed correctly #41

yads opened this issue Apr 10, 2015 · 1 comment
Labels

Comments

@yads
Copy link
Contributor

yads commented Apr 10, 2015

When you use the redis uri, such as :uri => 'redis://localhost:6379/0' The connection string seems to be parsed as {:type => 'redis', :host => 'localhost', :port => 6379, :path => '/0'}. This is then passed to the redis client. Unfortunately I think the redis client is getting confused with the :path parameter since that can specify a unix socket. It almost seems like for redis, you are better off passing the entire uri as the :url option to the redis client. (Note it doesn't matter if you specify the redis db in the uri or not)

FWIW, here's the code we are trying to use:

My::Application.configure do
  config.logstash = [{
                         :uri => 'redis://localhost:6379/0'
                     }, {
                         :type => :file
                     }]
end
@dwbutler
Copy link
Owner

Fixed in 0.10.1. Thanks for the contribution!

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

No branches or pull requests

2 participants