search

toyokazu / fluent-plugin-mqtt-io

Fluent plugin for MQTT Input/Output
Apache License 2.0
17 stars 12 forks source link

Syntax error in mqtt_proxy (missing semicolon) #15

Closed xeno010 closed 4 years ago

xeno010 commented 4 years ago

The last PR #14 is broken, a simicolon is missing.

see https://github.com/toyokazu/fluent-plugin-mqtt-io/blob/c8edafec8af59a073b588979a8026e75fee8f328/lib/fluent/plugin/mqtt_proxy.rb#L54-L60

Log:

Traceback (most recent call last):
        25: from /usr/bin/fluentd:23:in `<main>'
        24: from /usr/bin/fluentd:23:in `load'
        23: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/bin/fluentd:8:in `<top (required)>'
        22: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in `require'
        21: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in `require'
        20: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/command/fluentd.rb:330:in `<top (required)>'
        19: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/supervisor.rb:485:in `run_supervisor'
        18: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/engine.rb:79:in `run_configure'
        17: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/engine.rb:104:in `configure'
        16: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/root_agent.rb:146:in `configure'
        15: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/agent.rb:64:in `configure'
        14: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/agent.rb:64:in `each'
        13: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/agent.rb:74:in `block in configure'
        12: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/agent.rb:130:in `add_match'
        11: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/plugin.rb:109:in `new_output'
        10: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/plugin.rb:155:in `new_impl'
         9: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/registry.rb:44:in `lookup'
         8: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/registry.rb:99:in `search'
         7: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/registry.rb:99:in `each'
         6: from /usr/lib/ruby/gems/2.5.0/gems/fluentd-1.8.0/lib/fluent/registry.rb:102:in `block in search'
         5: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in `require'
         4: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in `require'
         3: from /usr/lib/ruby/gems/2.5.0/gems/fluent-plugin-mqtt-io-0.4.3/lib/fluent/plugin/out_mqtt.rb:4:in `<top (required)>'
         2: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:39:in `require'
         1: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:135:in `rescue in require'
/usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:135:in `require': /usr/lib/ruby/gems/2.5.0/gems/fluent-plugin-mqtt-io-0.4.3/lib/fluent/plugin/mqtt_proxy.rb:59: syntax error, unexpected tIDENTIFIER, expecting '}' (SyntaxError)
        keep_alive: @keep_alive
        ^~~~~~~~~~