-
Notifications
You must be signed in to change notification settings - Fork 978
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
Pattern TZ does not include most timezones #172
Comments
nor does it support British Summer Time (BST). |
It's missing lots of timezones because it is continental-US focused for whatever reason. So, do we try to add all known abbreviations for timezones? Or do we change TZ to be something like |
Here is a pretty comprehensive list of timezones: +1 to the |
I've made a pull request with this change: #235 but I'm not 100% sure it's the right thing to do... |
@robin13 |
This is a list of abbrev (scraped from timeanddate.com page above),
here is one regex...
@robin13 WDYT? |
I think we should create a MIL_TZ with |
Nice work @guyboertje ! :)
This would match nicely against |
I guess the final |
From: https://github.com/logstash-plugins/logstash-patterns-core/blob/master/patterns/grok-patterns
Doesn't include IST.
https://www.timeanddate.com/time/zones/ist
From:
TZ (?:[APMCE][SD]T|UTC)
To:
TZ (?:[APMCEI][SD]T|UTC)
The text was updated successfully, but these errors were encountered: