|Summary:||ASTERISK-08088: Restore autoframing option|
|Date Opened:||2006-11-08 15:33:09.000-0600||Date Closed:||2006-11-12 10:13:29.000-0600|
|Environment:||Attachments:||( 0) chan_sip-autoframing-fix.diff|
|Description:||Auto-Framing is a process by which Asterisk will honor the|
ptime: setting in the SDP. This feature was included in the
RTP packetization patch, but had issues as described below.
The feature was accidentily partially supressed in process_sdp()
due to the checks for unsupported SDP.
If debugging was not set, the code would work, but setting any
level of debug to validate the code would shortcut the proccessing
of the SDP.
****** ADDITIONAL INFORMATION ******
This feature defaults to off.
If enabled Asterisk will set the RTP packetization to
the size requested by the endpoint, creating symmetric
RTP packetization payloads.
This patch does the following:
- Removes the 'unsupported' check for ptime: with debugging set
- Allows peers/users to inherit and override the global autoframing option
-The original code gave undue priority to the global setting
If the global option and user/peer options were set to the same
value, it worked as expected, but if one was yes and the other
was no, then the results would not be what the administrator
- Adds Auto-Framing status display to:
sip show settings
sip show peer xxxx
sip show user xxxx
|Comments:||By: Olle Johansson (oej) 2006-11-10 07:23:42.000-0600|
+ ast_log(LOG_DEBUG, "Auto-Faming set \n");
Miss-spelled, don't log to DEBUG without option_debug set
Otherwise: Where do you parse the ptime option?
By: dea (dea) 2006-11-10 11:27:15.000-0600
That line can be dropped, it was part of instrumenting the code to
see what was happening. The ptime: option is parsed in process_sdp()
just below the test for sendrecv
By: Olle Johansson (oej) 2006-11-12 10:12:57.000-0600
Committed to svn 1.4 and trunk. 1.4 rev 47513, trunk rev 47514. Thank you for your contribution, DEA!