logstash-plugins / logstash-input-s3

Apache License 2.0
57 stars 150 forks source link

Remove commas from the s3 input config #154

Closed nephel closed 6 years ago

nephel commented 6 years ago

I tested with Logstash 6.3.1 and logstash-input-s3 3.3.5. If the commas are part of the config, I get the following error [2018-07-10T17:16:38,730][ERROR][logstash.agent ] Failed to execute action {:action=>LogStash::PipelineAction::Create/pipeline_id:main, :exception=>"LogStash::ConfigurationError", :message=>"Expected one of #, {, } at line 4, column 46 (byte 63) after input {\n \n s3 {\n \"access_key_id\" ...

without the commas it worked fine.

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

elasticsearch-bot commented 6 years ago

Rob Bavey merged this into the following branches!

Branch Commits
master 2cc73b6856f8395fd39aba2b65718cfb516bb3ba