[Home]

Summary:ASTERISK-17075: [patch] SIP calls to H323 Tandberg Endpoints are not fully answered
Reporter:Andy Boatman (mrhanman)Labels:
Date Opened:2010-12-07 09:26:17.000-0600Date Closed:2010-12-25 10:35:11.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Addons/chan_ooh323
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) Asterisk_console_incoming.log
( 1) Asterisk_console_outgoing.log
( 2) Asterisk_full_incoming.log
( 3) Asterisk_full_outgoing.log
( 4) Asterisk_to_Tandberg_fail.log
( 5) incoming_h323.log
( 6) ooh323.conf
( 7) outgoing_h323.log
( 8) Tandberg_1000_Incoming.log
( 9) Tandberg_1000_outgoing.log
(10) Tandberg_Gatekeeper_incoming.log
(11) Tandberg_Gatekeeper_outgoing.log
(12) tcs-msd-h245-2.patch
(13) tcs-msd-h245-3.patch
Description:When calling a Tandberg T1000 Endpoint from a SIP softphone, the Tandberg answers the call, but the SIP softphone continues to ring indefinitely.  Likewise, a call from the Tandberg endpoint to a SIP softphone continues to ring on the Tandberg after the call is answered on the SIP softphone.  Patch 2 from bug ASTERISK-1815401 has been applied to 1.8.0 sources.

****** ADDITIONAL INFORMATION ******

I've attached logs from calls going both ways.
Comments:By: Andy Boatman (mrhanman) 2010-12-07 09:30:58.000-0600

The same behavior results if calling the Endpoint directly, instead of going through the gateway.
By: Alexander Anikin (may213) 2010-12-11 15:53:53.000-0600

Hi,

pls attach h323_log here as in previous. I have some idea about this trouble but i need to see this log more accurately.
By: Andy Boatman (mrhanman) 2010-12-11 19:21:38.000-0600

How do I enable the log?  Since upgrading to 1.8, the log doesn't seem to have any information in it.  It gets created, but there is nothing in it.
By: Alexander Anikin (may213) 2010-12-12 05:26:05.000-0600

setup tracelevel=6 in /etc/asterisk/ooh323.conf, it's 1 (only error messages) by default.
By: Andy Boatman (mrhanman) 2010-12-13 09:32:42.000-0600

Ok, h323 logs are attached.
By: Alexander Anikin (may213) 2010-12-15 15:06:25.000-0600

Hi,
pls try attached patch. This patch change order of sending TCS and MSD packets (don't send MSD until TCS exchange procedure done). As in previous test please attach h323_log here.
By: Andy Boatman (mrhanman) 2010-12-16 08:55:48.000-0600

The patch works great for calls coming in from the Tandberg to Asterisk, but calls going from Asterisk to the Tandberg endpoint still do the same thing.  I attached the section of the h323_log with the problem.

Also, there was no video sent from the Tandberg to the SIP softphone.
By: Alexander Anikin (may213) 2010-12-16 11:29:00.000-0600

Video support isn't implemented for now. I have plan to do support for video calls but isn't easy task  and I don't know when I can.
Will see on attached log.
By: Andy Boatman (mrhanman) 2010-12-16 16:11:51.000-0600

Thanks for looking into this for me.  I really appreciate it.  If you need help testing video when the time comes, let me know.  I'd be happy to help.
By: Alexander Anikin (may213) 2010-12-16 16:54:13.000-0600

pls try new patch, it remove sending of starth245 facility message on outgoing calls, remote endpoint will begin h245 connection when he want.
By: Andy Boatman (mrhanman) 2010-12-17 10:08:12.000-0600

That seems to have fixed it.  I can now make calls to and from the Tandberg Endpoint.  Thanks so much for your help.
By: Alexander Anikin (may213) 2010-12-25 09:54:03.000-0600

Ok, thanks for testing, i'll close ticket.
By: Digium Subversion (svnbot) 2010-12-25 10:12:30.000-0600

Repository: asterisk
Revision: 299711

U   branches/1.8/addons/ooh323c/src/oochannels.c
U   branches/1.8/addons/ooh323c/src/ooh245.c
U   branches/1.8/addons/ooh323c/src/ooh323.c
U   branches/1.8/addons/ooh323c/src/ooq931.c

------------------------------------------------------------------------
r299711 | may | 2010-12-25 10:12:29 -0600 (Sat, 25 Dec 2010) | 20 lines

Change order of sending TCS and MSD packets

Change order of sending Terminal Capability Set and MasterSlave
Determination packets, MSD send when TCS exchange procedure is done
(we send tcs ack to remote and we have remote tcs ack already
or we receive tcs ack from remote and we have send our tcs ack to
remote  already). Some endpoints can work in this sequence only,
i suggest they can't work with both (tcs and msd) exchange procedures
simultaneously.

Also changed StartH245 facility message sending. It send on
incoming calls only due to some endpoints can't proccess properly
this facility messages on their incoming calls.

(issue ASTERISK-17075)
Reported by: MrHanMan
Patches:
     tcs-msd-h245-3.patch uploaded by may213 (license 454)
Tested by: MrHanMan, may213

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=299711
By: Digium Subversion (svnbot) 2010-12-25 10:31:44.000-0600

Repository: asterisk-addons
Revision: 1139

U   branches/1.6.2/channels/ooh323c/src/oochannels.c
U   branches/1.6.2/channels/ooh323c/src/ooh245.c
U   branches/1.6.2/channels/ooh323c/src/ooh323.c

------------------------------------------------------------------------
r1139 | may | 2010-12-25 10:31:44 -0600 (Sat, 25 Dec 2010) | 20 lines

Change order of sending TCS and MSD packets

Change order of sending Terminal Capability Set and MasterSlave
Determination packets, MSD send when TCS exchange procedure is done
(we send tcs ack to remote and we have remote tcs ack already
or we receive tcs ack from remote and we have send our tcs ack to
remote  already). Some endpoints can work in this sequence only,
i suggest they can't work with both (tcs and msd) exchange procedures
simultaneously.

Also changed StartH245 facility message sending. It send on
incoming calls only due to some endpoints can't proccess properly
this facility messages on their incoming calls.

(issue ASTERISK-17075)
Reported by: MrHanMan
Patches:
     tcs-msd-h245-3.patch uploaded by may213 (license 454)
Tested by: MrHanMan, may213

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk-addons?view=rev&revision=1139
By: Digium Subversion (svnbot) 2010-12-25 10:35:10.000-0600

Repository: asterisk
Revision: 299715

U   trunk/addons/ooh323c/src/oochannels.c
U   trunk/addons/ooh323c/src/ooh245.c
U   trunk/addons/ooh323c/src/ooh323.c
U   trunk/addons/ooh323c/src/ooq931.c

------------------------------------------------------------------------
r299715 | may | 2010-12-25 10:35:10 -0600 (Sat, 25 Dec 2010) | 20 lines

Change order of sending TCS and MSD packets

Change order of sending Terminal Capability Set and MasterSlave
Determination packets, MSD send when TCS exchange procedure is done
(we send tcs ack to remote and we have remote tcs ack already
or we receive tcs ack from remote and we have send our tcs ack to
remote  already). Some endpoints can work in this sequence only,
i suggest they can't work with both (tcs and msd) exchange procedures
simultaneously.

Also changed StartH245 facility message sending. It send on
incoming calls only due to some endpoints can't proccess properly
this facility messages on their incoming calls.

(closes issue ASTERISK-17075)
Reported by: MrHanMan
Patches:
     tcs-msd-h245-3.patch uploaded by may213 (license 454)
Tested by: MrHanMan, may213

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=299715