Summary: | ASTERISK-27124: app_playback.c:say_date_generic use timezonename parameter | ||
Reporter: | Holger Hans Peter Freyther (zecke) | Labels: | |
Date Opened: | 2017-07-11 04:51:33 | Date Closed: | 2017-08-16 06:49:32 |
Priority: | Major | Regression? | |
Status: | Closed/Complete | Components: | Applications/app_playback |
Versions: | 11.25.1 13.17.0 14.6.0 | Frequency of Occurrence | Constant |
Related Issues: | |||
Environment: | Attachments: | ||
Description: | say_date_generic receives a timezonename parameter but is not using it for ast_localtime. This is a problem when operating the system for users in different timezones. | ||
Comments: | By: Asterisk Team (asteriskteam) 2017-07-11 04:51:34.596-0500 Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution. A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report. Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process]. By: Holger Hans Peter Freyther (zecke) 2017-07-11 04:53:32.437-0500 I got a trivial patch and wait for the CLA to be accepted before uploading it to Gerrit. By: Rusty Newton (rnewton) 2017-07-12 14:38:40.487-0500 Thanks. I'll open up the issue. Once you submit to Gerrit the issue will be updated on the Gerrit Reviews tab. By: Rusty Newton (rnewton) 2017-07-12 14:39:37.652-0500 Be sure to edit the Affects Version field to reflect the relevant versions. By: Holger Hans Peter Freyther (zecke) 2017-07-12 16:23:43.515-0500 The issue seems to exist since app_playback.c was added. We have seen it with Asterisk11 but it might exist for longer. I have selected 11, 13 and 14. We require the fix to be in Asterisk14. https://gerrit.asterisk.org/#/c/6006/ and https://gerrit.asterisk.org/#/c/6005/ have been pushed into gerrit. |