logstash-logger 0.10.0 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 03eb27ccbc6770bb3348ffcedd6fdc46ba789731
-  data.tar.gz: 3d4d59a4c5a2b5ede515b44595c97fe73949d89d
+  metadata.gz: 951cad66dd80a5fab1d99deee38904ab91c8dc07
+  data.tar.gz: 65f9f5bb075bcb3586fef3dbebc3e29044de5529
 SHA512:
-  metadata.gz: 454217b66be5ceeaa97f904b0218d1c24651cb865c7cafb1bc69702025b90d8fe8ca3a5337b77296aea8dda1c2e78bce669348708f950195e30706436f8a7c0e
-  data.tar.gz: 3aa6228a59fbd9b540e5dc745caebebb2bfc11d65d16ad7265afd50a10710305e2cfc46ce1802118bbaf3fd8af25286b79c220efc93c7439c06924bb9a514735
+  metadata.gz: 3a75b6665961770c053c061fc5b1678e37e42c39372cd322cdeedcbc7e379bc3aa707bca6efd8bc551b1052a67ae148cf4bba6e416ed5d86f8d9a7ecac5b5f49
+  data.tar.gz: 08f67645cdde2de3a933773f8764de3e742aa3e3c52b8a693066a738f92d09d0334d214a2cdd378ff3059f3fe671ca4172d166e1043cbcebbc38e1fbf6407522

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.10.1
+- Fix for Redis URI parsing issue.
+Fixes [#41](https://github.com/dwbutler/logstash-logger/issues/41).
+Thanks [Vadim Kazakov](https://github.com/yads)!
+
 ## 0.10.0
 - Support for logging to Kafka.
 Fixes [#37](https://github.com/dwbutler/logstash-logger/issues/37).

data/README.md

@@ -415,6 +415,7 @@ logger = LogStashLogger.new('localhost', 5228, :tcp)
 * [Kurt Preston](https://github.com/KurtPreston)
 * [Chris Blatchley](https://github.com/chrisblatchley)
 * [Felix Bechstein](https://github.com/felixb)
+* [Vadim Kazakov](https://github.com/yads)
 
 ## Contributing
 

data/lib/logstash-logger/device/redis.rb

@@ -13,6 +13,9 @@ module LogStashLogger
       def initialize(opts)
         super
         @list = opts.delete(:list) || DEFAULT_LIST
+
+        normalize_path(opts)
+
         @redis_options = opts
 
         @batch_events = opts.fetch(:batch_events, 50)
@@ -21,6 +24,7 @@ module LogStashLogger
         buffer_initialize max_items: @batch_events, max_interval: @batch_timeout
       end
 
+
       def connect
         @io = ::Redis.new(@redis_options)
       end
@@ -64,6 +68,16 @@ module LogStashLogger
           end
         end
       end
+      
+      private
+
+      def normalize_path(opts)
+        path = opts.fetch(:path, nil)
+        if path
+          opts[:db] = path.gsub("/", "").to_i unless path.empty?
+          opts.delete(:path)
+        end
+      end
 
     end
   end

data/lib/logstash-logger/version.rb

@@ -1,3 +1,3 @@
 module LogStashLogger
-  VERSION = "0.10.0"
+  VERSION = "0.10.1"
 end

data/spec/device/redis_spec.rb

@@ -19,4 +19,34 @@ describe LogStashLogger::Device::Redis do
   it "defaults the Redis list to 'logstash'" do
     expect(redis_device.list).to eq('logstash')
   end
+
+  describe "initializer" do
+    let(:redis_options) { { host: HOST, port: 6379 } }
+    subject { LogStashLogger::Device::Redis.new(redis_options).connect }
+
+    context "path is not blank" do
+      before do
+        redis_options[:path] = "/0"
+      end
+
+      it "sets the db" do
+        expect(Redis).to receive(:new).with(hash_including(db: 0))
+        subject
+      end
+
+    end
+
+    context "path is blank" do
+      before do
+        redis_options[:path] = ""
+      end
+
+      it "does not set the db" do
+        expect(Redis).to receive(:new).with(hash_excluding(:db))
+        subject
+      end
+    end
+
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-logger
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.10.1
 platform: ruby
 authors:
 - David Butler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-22 00:00:00.000000000 Z
+date: 2015-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logstash-event