<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/1116/">https://reviewboard.asterisk.org/r/1116/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">
I was asked how to use this patch here is a DP example im using for testing ill be pushing some tweaks latter im happy with the way it is working now.the problems i was having turned out to be a format management issue in chan_local im now focusing on the switchover API hook
i dont want any code for switchover in channel.c it must be in the channel drivers i have put a patch for T38SWITCHOVER option in sip that needs some work.
There is now no dedicated app for T38Gateway and there is no hijacking of the core bridge loop.
the gateway hooks into the core loop and replaces T.38 negotiation only if set using FAXOPT this is done entirely inside res_fax.
[faxtest]
;This can be set as a default in res_fax.conf
exten => _X.,1,SET(FAXOPT(t38gateway)=yes)
;Dial into local T.30 for testing i have removed T38 support from local for this [no queryoption] can use the F option in trunk
exten => _X.,n(out),Dial(Local/${EXTEN}@faxin)
exten => _X.,n,Hangup
[faxin]
exten => _X.,1,Answer()
exten => _X.,n,Wait(3)
exten => _X.,n,ReceiveFAX(/var/spool/asterisk/fax/${CDR(linkedid)}.tiff)
== Using UDPTL CoS mark 5
== Using SIP RTP CoS mark 5
-- Executing [0803@faxtest:1] Set("SIP/0819-0000000b", "FAXOPT(t38gateway)=yes") in new stack
-- Executing [0803@faxtest:2] Dial("SIP/0819-0000000b", "Local/0803@faxin") in new stack
-- Executing [0803@faxin:1] Answer("Local/0803@faxin-1d58;2", "") in new stack
-- Called 0803@faxin
-- Local/0803@faxin-1d58;1 answered SIP/0819-0000000b
-- Executing [0803@faxin:2] Wait("Local/0803@faxin-1d58;2", "3") in new stack
*****
We have changed to ALAW from G.729 the patch in chan_local will wake up and fix this we cannot use narrowband here
*****
[Mar 15 09:09:44] NOTICE[27453]: channel.c:4071 __ast_read: Dropping incompatible voice frame on Local/0803@faxin-1d58;2 of format g729 since our native format has changed to 0x8 (alaw)
-- Running Gateway activestate=4 (SIP/0819-0000000b) and inactivestate=0 (Local/0803@faxin-1d58;1)
[Mar 15 09:09:44] NOTICE[27452]: udptl.c:1076 ast_udptl_write: (SIP/0819): UDPTL Transmission error to 192.168.150.155:16428: Operation not permitted
-- Executing [0803@faxin:3] ReceiveFAX("Local/0803@faxin-1d58;2", "/var/spool/asterisk/fax/1300172981.17.tiff") in new stack
-- Channel 'Local/0803@faxin-1d58;2' receiving FAX '/var/spool/asterisk/fax/1300172981.17.tiff'
== Spawn extension (faxtest, 0803, 2) exited non-zero on 'SIP/0819-0000000b'
-- Connection Statistics
Bit Rate :14400
ECM : No
Pages : 1</pre>
<br />
<p>- irroot</p>
<br />
<p>On March 9th, 2011, 10:26 a.m., irroot wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Russell Bryant, dimas, Leif Madsen, and dafe_von_cetin.</div>
<div>By irroot.</div>
<p style="color: grey;"><i>Updated 2011-03-09 10:26:09</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi there the patch that was going around circa 2008 to implement this in 1.4/1.6 app_fax has been moved to trunk [1.10]
ive made some cleanups and moved it into res_fax res_fax_spandsp this is the framework and not production code
unfortunately i have no means of testing it at the moment and require help.
i have cleaned the code up substantially it is related to R459
hope this is found useful and aids in the goal to get it in 1.10.
Adds application FaxGateway / FaxDetect
Adds alternate bridge to Dial with new option.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/view.php?id=13405">13405</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/apps/app_faxdetect.c <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/channels/chan_sip.c <span style="color: grey">(310135)</span></li>
<li>/trunk/channels/sip/include/sip.h <span style="color: grey">(310135)</span></li>
<li>/trunk/configs/res_fax.conf.sample <span style="color: grey">(310135)</span></li>
<li>/trunk/configs/sip.conf.sample <span style="color: grey">(310135)</span></li>
<li>/trunk/include/asterisk/channel.h <span style="color: grey">(310135)</span></li>
<li>/trunk/include/asterisk/res_fax.h <span style="color: grey">(310135)</span></li>
<li>/trunk/main/channel.c <span style="color: grey">(310135)</span></li>
<li>/trunk/res/res_fax.c <span style="color: grey">(310135)</span></li>
<li>/trunk/res/res_fax_spandsp.c <span style="color: grey">(310135)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1116/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>