nsinnovations/fusionpbx
2
0
Fork 0
mirror of https://github.com/fusionpbx/fusionpbx.git synced 2025-12-30 00:53:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix: Ensure proper absolute_codec_string for fax_queue (#7359)

Browse Source 
The absolute_codec_string parameter for FreeSWITCH was being incorrectly
for formatted, causing issues with fax transmissions, particularly in
regions preferring the PCMA codec.
Without proper quoting, the comma-separated codec list PCMU,PCMA was
being parsed incorrectly by FreeSWITCH. The second codec PCMA was
misinterpreted as a separate dial string parameter, resulting in
absolute_codec_string being set to only PCMU. This led to negotiation
failures when the receiving end required PCMA.
This commit resolves this by enclosing the codec list within *escaped*
single quotes \'PCMU,PCMA\'. **Crucially, without this escaping,
FreeSWITCH would not correctly interpret the quoted string.** The
literal quotes are necessary to ensure that PCMU,PCMA is treated as a
single value. By implementing this change, we guarantee that both PCMU
and PCMA are correctly included in the codec negotiation, resolving the
fax transmission failures in PCMA-dominant regions.
This commit is contained in:
hacc1225
2025-08-15 15:29:27 +02:00
committed by GitHub
parent 57b26b9d44
commit 1fc0962de2
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/fax_queue/resources/job/fax_send.php
View File

@@ -394,7 +394,7 @@
//$dial_string .= "fax_retry_sleep=180,";
$dial_string .= "fax_verbose=true,";
//$dial_string .= "fax_use_ecm=off,";
$dial_string .= "absolute_codec_string=PCMU,PCMA,";
$dial_string .= "absolute_codec_string=\'PCMU,PCMA\',";
$dial_string .= "api_hangup_hook='lua app/fax/resources/scripts/hangup_tx.lua'";
//connect to event socket and send the command