[asterisk-dev] [Code Review] 4320: res_fax: Make T.38 negotiation timeout configurable and handle T.38 switch failure

[opticron]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4320/
-----------------------------------------------------------

(Updated Jan. 7, 2015, 7:25 p.m.)


Review request for Asterisk Developers.


Changes
-------

Address Richard's comments.


Repository: Asterisk


Description
-------

This change makes the T.38 negotiation timeout configurable via res_fax.conf or the FAXOPT() dialplan function. It was previously hard coded to be 5 seconds.

This change also handles T.38 switch failures by aborting the fax since in the case where this can happen, both sides have agreed to switch to T.38 and Asterisk is unable to do so.


Diffs (updated)
-----

  branches/11/res/res_fax.c 430372 
  branches/11/include/asterisk/res_fax.h 430372 
  branches/11/configs/res_fax.conf.sample 430372 

Diff: https://reviewboard.asterisk.org/r/4320/diff/


Testing
-------

Manual testing and the test in review 4321.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150108/7cfc68ef/attachment-0001.html>