-
Notifications
You must be signed in to change notification settings - Fork 150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix a few issues regarding the s3 input #16
Conversation
ph
commented
Feb 17, 2015
- Allow the user to define where the plugin should keep his temporary files
- Bugfix, If a S3 object key exactly match the prefix we will skip it fixes Setting up a prefix in logstash make him treat it like an actual file #10
- Use info instead of error when using a user defined sincedb_path
- adding integration test
- Move back the GZIP file and the normal file logic back into the plugin.
@purbon I wish I had your rspec presentation before writing this. Could you review/test? |
Jajaja, will do! |
@purbon review? |
@suyograo will do, will do, hope to do it today. |
We will need to yank LogStash::Codecs::GzipLines and LogStash::Codecs::Cloudfront and remove the two repositories since I have moved back the functionality into the S3 input. https://github.com/logstash-plugins/logstash-codec-cloudfront and https://github.com/logstash-plugins/logstash-codec-gzip_lines |
@ph reviewing |
else | ||
decorate(event) | ||
|
||
event["cloudfront_version"] = metadata[:cloudfront_version] unless metadata[:cloudfront_version].nil? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would avoid as much as possible the usage of unless, is more clean when you use if not. But this is a personal style test.
* Allow the user to define where the plugin should keep his temporary files * Bugfix, If a S3 object key exactly match the prefix we will skip it fixes logstash-plugins#10 * Use info instead of error when using a user defined sincedb_path adding integration test
4fb0023
to
6c47c77
Compare
Merged sucessfully into master! |