[asterisk-bugs] [Asterisk 0012380]: [patch] T.38 support for chan_h323 with t.38 - t30 conversion

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12380 
====================================================================== 
Reported By:                may213
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12380
Category:                   Channels/NewFeature
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 112599 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             04-07-2008 19:33 CDT
Last Modified:              04-08-2008 15:10 CDT
====================================================================== 
Summary:                    [patch] T.38 support for chan_h323 with t.38 - t30
conversion
Description: 
Hello!

There is patch for h323 channel code which enable t.38 proto support,
translate t.38 packets to alaw sound packets. 
====================================================================== 

---------------------------------------------------------------------- 
 sergee - 04-08-08 15:10  
---------------------------------------------------------------------- 
may213,

First of all new features should be adjusted to trunk, and not to 1.4
branch. Second, i believe that most part of Asterisk developers are using
uLaw - not aLaw as we do :) so i think new feature should support both
modes natively.
Next thing - this patch depends on SpanDSP, so it can't be accepted into
asterisk, due to dual-licensing issues. BUT! It can be accepted into
asterisk-addons.

Here is a roadmap to get this feature in asterisk-addons (as i see it,
maybe someone will correct me):

1. Put hooks for t28 processing (converting to *Law) inside of h323
2. Add support for ulaw
3. Put all code related to spanDSP inside asterisk-addons (e.g. inside
app_fax, or may be res_fax etc..)


Also, i'm not sure that this feature is good from architectural point of
view. I believe that we need some general facility for relaying fax between
different channels, not some local hacks. But that's a long story, and your
code is already works. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
04-08-08 15:10  sergee         Note Added: 0085156                          
======================================================================