[Asterisk-code-review] cdr syslog: Deprecate unmaintained module (asterisk[master])

Sean Bright asteriskteam at digium.com
Wed Jan 10 08:03:34 CST 2018


Hello Jenkins2, Corey Farrell, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/7901

to look at the new patch set (#3).

Change subject: cdr_syslog: Deprecate unmaintained module
......................................................................

cdr_syslog: Deprecate unmaintained module

There has been an open issue against cdr_syslog (ASTERISK~14441) about
a race condition for 7.5 years that has never been addressed. Because
this module is effectively unmaintained and currently broken, there is
no sense in keeping it around.

If logging CDRs to syslog is a desirable feature, it would probably be
better to write the logs directly to the syslog server via socket
instead of using the facilities provided by openlog/syslog/closelog.
Doing so would address the race condition referenced in the associated
issue.

Change-Id: Ic77b94cd97f355a9cf5b1d3f3444964a6e0ba5dc
---
M CHANGES
M UPGRADE.txt
M cdr/cdr_syslog.c
3 files changed, 11 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/01/7901/3
-- 
To view, visit https://gerrit.asterisk.org/7901
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic77b94cd97f355a9cf5b1d3f3444964a6e0ba5dc
Gerrit-Change-Number: 7901
Gerrit-PatchSet: 3
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180110/9c4607da/attachment-0001.html>


More information about the asterisk-code-review mailing list