fluentd 1.6.1-x64-mingw32 → 1.6.2-x64-mingw32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 53f0e1fcbe42769db859130cc65d68573214999990006f686b26ddc9fde8bd50
-  data.tar.gz: c024d98b107ad690bb2375848b1b7342f043967a09779f16ca043bf43fc24843
+  metadata.gz: 87303e3a8f368b5f5c7ca75d8cfba43a5fe52e84ff5ff455111bdcc750ea3f8a
+  data.tar.gz: b16f78a81686875a87f401471ebeab06bd2d6deb39baf422c823fd9b3ac8cd30
 SHA512:
-  metadata.gz: e17dd1c0eb93b91a0d43fc785e62a61db892c2f9c805d5afc29a74b08c19ab9a3a201636a3e5fc7b63b9c6fddc490caa3f4d05f7f5359a5dc1ba92e3e9e631ae
-  data.tar.gz: 4253c6734fbc94fb5475740751cde64f0a172e0560511c6c8641e1c056dc1ae9c5d71bf05dac6b93e2b7f8f221d991b283e959ed5d09308a0d2151a654c186c8
+  metadata.gz: 0a0676a4eef431b603a2c3117fe90f588f544bf53fdc9eece96112fc19ad1179bb6dd4261db783d29b913e48e6f7624e2b524356cd010535bee425664cfbfc5a
+  data.tar.gz: 6abb329408250d0dcce520762e68ed8845921f9b15f22383845879c7e4a7754d39a4d94231a7ba54dba620a99bec55fa85e4699d7555e5c2f44110a535b8047a

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # v1.6
 
+## Release v1.6.2 - 2019/07/11
+
+### Bug fixes
+
+* http_server helper: Add title argument to support multiple servers
+  https://github.com/fluent/fluentd/pull/2493
+
 ## Release v1.6.1 - 2019/07/10
 
 ### Enhancement

data/lib/fluent/plugin/in_monitor_agent.rb

@@ -208,7 +208,7 @@ module Fluent::Plugin
 
       log.debug "listening monitoring http server on http://#{@bind}:#{@port}/api/plugins for worker#{fluentd_worker_id}"
       api_handler = APIHandler.new(self)
-      create_http_server(addr: @bind, port: @port, logger: log, default_app: NotFoundJson) do |serv|
+      create_http_server(:in_monitor_http_server_helper, addr: @bind, port: @port, logger: log, default_app: NotFoundJson) do |serv|
         serv.get('/api/plugins') { |req| api_handler.plugins_ltsv(req) }
         serv.get('/api/plugins.json') { |req| api_handler.plugins_json(req) }
         serv.get('/api/config') { |req| api_handler.config_ltsv(req) }

data/lib/fluent/plugin_helper/http_server.rb

@@ -34,11 +34,12 @@ module Fluent
       # close    : correct stopped threads
       # terminate: kill thread
 
+      # @param title [Symbol] the thread name. this value should be unique.
       # @param addr [String] Listen address
       # @param port [String] Listen port
       # @param logger [Logger] logger used in this server
       # @param default_app [Object] This method must have #call.
-      def create_http_server(addr:, port:, logger:, default_app: nil)
+      def create_http_server(title, addr:, port:, logger:, default_app: nil)
         unless block_given?
           raise ArgumentError, 'BUG: callback not specified'
         end
@@ -48,7 +49,7 @@ module Fluent
         end
 
         _block_until_http_server_start do |notify|
-          thread_create(:plugin_helper_http_server) do
+          thread_create(title) do
             @_http_server.start(notify)
           end
         end

data/lib/fluent/version.rb

@@ -16,6 +16,6 @@
 
 module Fluent
 
-  VERSION = '1.6.1'
+  VERSION = '1.6.2'
 
 end

data/test/plugin_helper/test_http_server_helper.rb

@@ -50,7 +50,8 @@ class HtttpHelperTest < Test::Unit::TestCase
   %w[get head].each do |n|
     define_method(n) do |uri, header = {}|
       url = URI.parse(uri)
-      req = Net::HTTP.const_get(n.capitalize).new(url, header)
+      headers = { 'Content-Type' => 'application/x-www-form-urlencoded/' }.merge(header)
+      req = Net::HTTP.const_get(n.capitalize).new(url, headers)
       Net::HTTP.start(url.host, url.port) do |http|
         http.request(req)
       end
@@ -60,7 +61,8 @@ class HtttpHelperTest < Test::Unit::TestCase
   %w[post put patch delete options trace].each do |n|
     define_method(n) do |uri, body = '', header = {}|
       url = URI.parse(uri)
-      req = Net::HTTP.const_get(n.capitalize).new(url, header)
+      headers = { 'Content-Type' => 'application/x-www-form-urlencoded/' }.merge(header)
+      req = Net::HTTP.const_get(n.capitalize).new(url, headers)
       req.body = body
       Net::HTTP.start(url.host, url.port) do |http|
         http.request(req)
@@ -71,7 +73,7 @@ class HtttpHelperTest < Test::Unit::TestCase
   sub_test_case 'Create a HTTP server' do
     test 'monunt given path' do
       on_driver do |driver|
-        driver.create_http_server(addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
+        driver.create_http_server(:http_server_helper_test, addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
           s.get('/example/hello') { [200, { 'Content-Type' => 'text/plain' }, 'hello get'] }
           s.post('/example/hello') { [200, { 'Content-Type' => 'text/plain' }, 'hello post'] }
           s.head('/example/hello') { [200, { 'Content-Type' => 'text/plain' }, 'hello head'] }
@@ -106,7 +108,7 @@ class HtttpHelperTest < Test::Unit::TestCase
 
     test 'when path does not start with `/` or ends with `/`' do
       on_driver do |driver|
-        driver.create_http_server(addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
+        driver.create_http_server(:http_server_helper_test, addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
           s.get('example/hello') { [200, { 'Content-Type' => 'text/plain' }, 'hello get'] }
           s.get('/example/hello2/') { [200, { 'Content-Type' => 'text/plain' }, 'hello get'] }
         end
@@ -121,7 +123,7 @@ class HtttpHelperTest < Test::Unit::TestCase
 
     test 'when error raised' do
       on_driver do |driver|
-        driver.create_http_server(addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
+        driver.create_http_server(:http_server_helper_test, addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
           s.get('/example/hello') { raise 'error!' }
         end
 
@@ -132,7 +134,7 @@ class HtttpHelperTest < Test::Unit::TestCase
 
     test 'when path is not found' do
       on_driver do |driver|
-        driver.create_http_server(addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
+        driver.create_http_server(:http_server_helper_test, addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
           s.get('/example/hello') { [200, { 'Content-Type' => 'text/plain' }, 'hello get'] }
         end
 
@@ -143,7 +145,7 @@ class HtttpHelperTest < Test::Unit::TestCase
 
     test 'params and body' do
       on_driver do |driver|
-        driver.create_http_server(addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
+        driver.create_http_server(:http_server_helper_test, addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do |s|
           s.get('/example/hello') do |req|
             assert_equal(req.query_string, nil)
             assert_equal(req.body, nil)
@@ -193,7 +195,7 @@ class HtttpHelperTest < Test::Unit::TestCase
         end
 
         stub(Fluent::PluginHelper::HttpServer::Server).new(anything) { server }
-        driver.create_http_server(addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do
+        driver.create_http_server(:http_server_helper_test, addr: '127.0.0.1', port: PORT, logger: NULL_LOGGER) do
           # nothing
         end
         driver.stop

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluentd
 version: !ruby/object:Gem::Version
-  version: 1.6.1
+  version: 1.6.2
 platform: x64-mingw32
 authors:
 - Sadayuki Furuhashi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-11 00:00:00.000000000 Z
+date: 2019-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: msgpack