[asterisk-dev] [Code Review] Fax Gateway Implementation T30<->T38

[irroot]

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

(Updated 2011-05-09 03:26:02.432931)


Review request for Asterisk Developers, Russell Bryant, Matthew Nicholson, dimas, Leif Madsen, and dafe_von_cetin.


Changes
-------

Ok got me some access to the office and sorted out the send params on CED so here it is ...

Got some customers testing it now with HylaFax AKA "Print FAX to PBX/Asterisk"

    -- Executing [s at macro-configdial:38] Dial("OOH323/10.0.0.50-0", "mISDN/g:out/XXXXXXXXXX/e128,,WgT") in new stack
    -- Called g:out/XXXXXXXXXXX/e128
    -- mISDN/1-u2 is proceeding passing it to OOH323/10.0.0.50-0
    -- mISDN/1-u2 is ringing
    -- mISDN/1-u2 answered OOH323/10.0.0.50-0
    -- T.38 Gateway starting for chan OOH323/10.0.0.50-0 and peer mISDN/1-u2
    -- Connection Statistics
        Bit Rate :9600
        ECM : Yes
        Pages : 1

Greg


Summary
-------

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.


This addresses bugs 13405 and 19215.
    https://issues.asterisk.org/view.php?id=13405
    https://issues.asterisk.org/view.php?id=19215


Diffs (updated)
-----

  /trunk/apps/app_faxdetect.c PRE-CREATION 
  /trunk/channels/chan_sip.c 318105 
  /trunk/channels/sip/include/sip.h 318105 
  /trunk/include/asterisk/res_fax.h 318105 
  /trunk/main/channel.c 318105 
  /trunk/res/res_fax.c 318105 
  /trunk/res/res_fax_spandsp.c 318105 

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


Testing
-------


Thanks,

irroot

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110509/af1428e4/attachment.htm>