-
Notifications
You must be signed in to change notification settings - Fork 56
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 issue related to Error TZInfo::AmbiguousTime #70
base: main
Are you sure you want to change the base?
Conversation
…3+00:00 is an ambiguous local time See related issues : - logstash-plugins/logstash-input-jdbc#121 - tzinfo/tzinfo#32
💚 CLA has been signed |
@spawnrider thank you! You saved our day, I applied the patch to our image and it worked. Please sign the CLA to go upstream :) |
I just signed the CLA. How can we go ahead with this PR ? |
hello, any news about integrating this PR ? |
Hi, In the meantime, the best thing for you, is to build your own lib version and inject it into Logstash. |
I have encountered the same issue, any news on when this PR is going to be merged? |
Same here, I am patching the file manually at every new version, but I wish
Logstash included the proposed solution.
|
Hello, any update? |
Fix issue related to Error: TZInfo::AmbiguousTime: 2015-10-25T02:10:03+00:00 is an ambiguous local time
See related issues :
Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/