Ticket #2560 (closed Bugs: fixed)

Opened 5 years ago

Last modified 5 years ago

1.4 deprecated option warning: notransfer in iax_additional.conf

Reported by: habile Assigned to: p_lindheimer
Priority: minor Milestone: 2.4
Component: Core - Users/Devices Version: 2.4-branch
Keywords: Cc:
Confirmation: Need testing Distro:
Backend Engine: All Distro Ver:
Backend Ver: SVN Revision (if applicable):

Description

The 'notransfer=yes' option written to iax_additional.conf generates the following warning on 1.4.x

[Dec 15 20:14:18] NOTICE[32651] chan_iax2.c: The option 'notransfer' is deprecated in favor of 'transfer' which has options 'yes', 'no', and 'mediaonly'

Attachments

iax_transfer.diff (2.7 kB) - added by pnlarsson on 01/01/08 05:44:55.
A better way
notransfer.patch (2.3 kB) - added by p_lindheimer on 01/01/08 17:10:13.
convert notransfer to transfer - alternative

Change History

12/15/07 15:33:26 changed by habile

Additional deprecated messages - separate tickets?

[Dec 15 20:36:49] WARNING[4105] chan_sip.c: tos value at line 13 is deprecated. See doc/ip-tos.txt for more information. [Dec 15 20:36:49] WARNING[4105] chan_skinny.c: Option 'port' at line 5 of skinny.conf has been deprecated. Please use 'bindport' instead. [Dec 15 20:36:49] NOTICE[4105] app_queue.c: The 'monitor-join' queue option is deprecated. Please use monitor-type=mixmonitor instead.

12/16/07 21:14:21 changed by p_lindheimer

  • confirmation changed from Unreviewed to Confirmed.
  • version deleted.
  • milestone changed from Cut Line to 2.4.

12/29/07 18:20:35 changed by pnlarsson

r5558 - fixed sip and iax tos

12/31/07 19:31:38 changed by pnlarsson

Attached a patch for fixing the transfer stuff with iax, it's a bit crude - it's still notransfer in the GUI - but how shall we change that?

01/01/08 05:44:55 changed by pnlarsson

  • attachment iax_transfer.diff added.

A better way

01/01/08 05:47:46 changed by pnlarsson

An updated patch that also solves the allow / disallow warnings. I like this method better than doing it the SQL way #2574.

01/01/08 05:49:30 changed by pnlarsson

And media should of course be mediaonly and nothing else...

01/01/08 16:54:05 changed by p_lindheimer

I had the attached patch sitting in svn un-checked in because I had not tested it. I should have said something so work was not duplicated. I structured the patch in a switch statement for the new options so that it could deal with other options. (For example the allow/disallow could be put into the switch statement if we want to address these). I'll paste it in here for review so we can decided which way to deal with this. See attached.

01/01/08 17:10:13 changed by p_lindheimer

  • attachment notransfer.patch added.

convert notransfer to transfer - alternative

01/06/08 13:40:44 changed by p_lindheimer

  • confirmation changed from Confirmed to Need testing.
  • version set to 2.4-branch.

r5563 - needs testing, would be nice before I publish this if anyone has a chance to look at it today.

01/12/08 16:24:45 changed by p_lindheimer

  • status changed from new to closed.
  • resolution set to fixed.