Skip to content

Conversation

@ph
Copy link
Contributor

@ph ph commented Jun 22, 2016

Backport of pr #7 for 2.X

ph added 2 commits June 21, 2016 23:20
This PR include the following Refactor:
  - Move from `Kernel.stop` to `Open3.popen3`
  - When logstash is running in debug mode the stdout and the stderr of
  the plugin will be streamed to the log.
  - Added an option to run the command in `quiet` mode so the result of
  the command is not directly print to stdout.
  - Added tests

Fixes: #4
@elasticsearch-bot
Copy link

Pier-Hugues Pellerin merged this into the following branches!

Branch Commits
plugin-api-v1 4928ece, 51ca1b4

elasticsearch-bot pushed a commit that referenced this pull request Jun 22, 2016
…ve from `Kernel.stop` to `Open3.popen3` - When logstash is running in debug mode the stdout and the stderr of the plugin will be streamed to the log. - Added an option to run the command in `quiet` mode so the result of the command is not directly print to stdout. - Added tests

Fixes: #4

Fixes #9
elasticsearch-bot pushed a commit that referenced this pull request Jun 22, 2016
@ph ph closed this Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants