-
Notifications
You must be signed in to change notification settings - Fork 468
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
SnmpTrapdParser: Fix tm_hour differences in outgoing messages came from snmptrapd() source #1746
SnmpTrapdParser: Fix tm_hour differences in outgoing messages came from snmptrapd() source #1746
Conversation
Build SUCCESS, the tests were executed on test branch: master and test suite: functions |
@kira-syslogng retest this please test branch=micek_timezone_for_ose; |
Build FAILURE, the tests were executed on test branch: micek_timezone_for_ose and test suite: functions |
bbb05ce
to
fd7d85f
Compare
Build FAILURE, the tests were executed on test branch: micek_timezone_for_ose and test suite: functions |
fd7d85f
to
558731e
Compare
Build FAILURE, the tests were executed on test branch: micek_timezone_for_ose and test suite: functions |
558731e
to
e3dd143
Compare
Build FAILURE, the tests were executed on test branch: micek_timezone_for_ose and test suite: functions |
e3dd143
to
4b9e1a6
Compare
Build SUCCESS, the tests were executed on test branch: master and test suite: functions |
…om snmptrapd() source * The difference happened when the date of the incoming message is in summer time, and the localtime is in winter time. Timezone: Europe/Budapest * Example date from incoming message: 2017-05-11 16:07:17 * Wrong date in outgoing message: 2017-05-11 17:07:17 * Fixed date in outgoing message: 2017-05-11 16:07:17 * Added unit test fixes Signed-off-by: Andras Mitzki <andras.mitzki@balabit.com>
4b9e1a6
to
ceacc93
Compare
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.
Nice catch. Setting isdst to -1 in these cases is not really documented and took a few years to get right in the syslogformat module as well.
Build SUCCESS, the tests were executed on test branch: master and test suite: functions |
This PR fixes a one hour difference when a incoming timestamp is in DST and a local time is in winter time, and the local timezone is not CEST.
The issue can be reproduced with snmptrapd-source.
Signed-off-by: Andras Mitzki andras.mitzki@balabit.com
Signed-off-by: Gabor Nagy gabor.nagy@balabit.com