[Asterisk-code-review] main/cdr: Fixed cdr start overwriting (asterisk[master])

Joshua C. Colp asteriskteam at digium.com
Wed Dec 19 04:40:46 CST 2018


Joshua C. Colp has posted comments on this change. ( https://gerrit.asterisk.org/10747 )

Change subject: main/cdr: Fixed cdr start overwriting
......................................................................


Patch Set 4: Code-Review+1

I've run this through all the tests, done manual testing of scenarios, and also become familiar with the CDR implementation itself. This change is the correct fix for this, and fixes origination involving a Local channel to comply with the CDR specification itself as well. I wasn't able to observe any other changes as a result of this.

This change should be cherry picked to the other appropriate branches.


-- 
To view, visit https://gerrit.asterisk.org/10747
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I921bc04064b6cff1deb2eea56a94d86489561cdc
Gerrit-Change-Number: 10747
Gerrit-PatchSet: 4
Gerrit-Owner: sungtae kim <pchero21 at gmail.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Reviewer: Joshua C. Colp <jcolp at digium.com>
Gerrit-Reviewer: sungtae kim <pchero21 at gmail.com>
Gerrit-Comment-Date: Wed, 19 Dec 2018 10:40:46 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181219/2ff0f68e/attachment.html>


More information about the asterisk-code-review mailing list