[Asterisk-code-review] app_voicemail: Fix missing email in msg_create_from_file. (asterisk[master])

N A asteriskteam at digium.com
Tue Nov 8 14:40:50 CST 2022


Attention is currently required from: N A.

Hello Joshua Colp, Friendly Automation, 

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

    https://gerrit.asterisk.org/c/asterisk/+/19468

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

Change subject: app_voicemail: Fix missing email in msg_create_from_file.
......................................................................

app_voicemail: Fix missing email in msg_create_from_file.

msg_create_from_file currently does not dispatch emails,
which means that applications using this function, such
as MixMonitor, will not trigger notifications to users
(only AMI events are sent our currently). This is inconsistent
with other ways users can receive voicemail.

This is fixed by adding an option that attempts to send
an email and falling back to just the notifications as
done now if that fails. The existing behavior remains
the default.

ASTERISK-30283 #close

Change-Id: I597cbb9cf971a18d8776172b26ab187dc096a5c7
---
M apps/app_voicemail.c
M configs/samples/voicemail.conf.sample
A doc/CHANGES-staging/app_voicemail_attachext.txt
3 files changed, 59 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/68/19468/2
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/19468
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I597cbb9cf971a18d8776172b26ab187dc096a5c7
Gerrit-Change-Number: 19468
Gerrit-PatchSet: 2
Gerrit-Owner: N A <asterisk at phreaknet.org>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: N A <asterisk at phreaknet.org>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221108/84165ecd/attachment.html>


More information about the asterisk-code-review mailing list