| Summary: | ASTERISK-27080: res_pjsip_t38: Slow T.38 re-invite rejection if remote leg has T.38 disabled | ||||
| Reporter: | Torrey Searle (tsearle) | Labels: | pjsip | ||
| Date Opened: | 2017-06-22 07:42:59 | Date Closed: | 2017-08-16 06:46:49 | ||
| Priority: | Major | Regression? | |||
| Status: | Closed/Complete | Components: | Resources/res_pjsip_t38 | ||
| Versions: | 13.14.1 | Frequency of Occurrence | |||
| Related Issues: |
| ||||
| Environment: | Attachments: | ||||
| Description: | When a T.38 enabled endpoint receives a T.38 re-invite and tries to forward it to a pjsip endpoint that has T.38 disabled. The AST_T38_REQUEST_NEGOTIATE control frame is discarded by the second leg instead of responding with a AST_T38_REFUSED. This causes the a leg to wait 5 seconds before timing out and responding with a 488. | ||||
| Comments: | By: Asterisk Team (asteriskteam) 2017-06-22 07:43:00.449-0500 Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution. A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report. Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process]. By: Alexander Traud (traud) 2018-06-04 02:29:35.268-0500 In the course of this, you added a test case to the Test Suite, which uses a command tool called ‘rtpdump’. The test does not explain where to find and how to compile that tool. The [Wiki entry|http://wiki.asterisk.org/wiki/display/AST/Installing+the+Asterisk+Test+Suite] about the Test Suite was not updated either. I found a [rtpdump|http://www.cs.columbia.edu/IRT/software/rtptools/] but that gives: {{rtpdump: invalid option \-\- 'd'}}. Therefore, [~tsearle], please, elaborate which tool you use(d), so this can be added to the Wiki. By: Friendly Automation (friendly-automation) 2018-07-10 06:56:10.140-0500 Change 9333 merged by Joshua Colp: res_pjsip_t38.c: Be smarter about how we respond when T.38 is disabled. [https://gerrit.asterisk.org/9333|https://gerrit.asterisk.org/9333] By: Friendly Automation (friendly-automation) 2018-07-10 07:23:02.510-0500 Change 9335 merged by Joshua Colp: res_pjsip_t38.c: Be smarter about how we respond when T.38 is disabled. [https://gerrit.asterisk.org/9335|https://gerrit.asterisk.org/9335] By: Friendly Automation (friendly-automation) 2018-07-10 07:23:31.880-0500 Change 9334 merged by Joshua Colp: res_pjsip_t38.c: Be smarter about how we respond when T.38 is disabled. [https://gerrit.asterisk.org/9334|https://gerrit.asterisk.org/9334] | ||||