Asterisk
  1. Asterisk
  2. ASTERISK-5281

[patch] videosupport option in sip.conf should not be global

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Labels:
      None
    • Mantis ID:
      5427
    • Regression:
      No

      Description

      Currently, videosupport option can only be set in the [general] section.
      There are many broken gateways out there that do not handle SDP's which contain video. Some Cisco implementation will simply send back a 183 Session Progress with a Content-Type: application/sdp\r\n and a Content-Length: 0\r\n when they see a m=video in the INVITE (see issue 0004002).
      As a result, Asterisk outputs:
      process_sdp: Insufficient information for SDP (m = '', c = ''), the call goes through but there is no audio. Audio works fine if videosupport is off and the INVITE is sent without video.
      Setting videosupport=no only for those peers that are known to exhibit such bad behaviour would be helpful.

        Activity

        Hide
        Olle Johansson added a comment -

        For RTCP, please vist bug ASTERISK-2815 where work is going on. The patch in there was updated today.

        For other video functions, we will open another report/branch. Feel free to submit your own additions to a bug report, and we'll be on our way!

        Show
        Olle Johansson added a comment - For RTCP, please vist bug ASTERISK-2815 where work is going on. The patch in there was updated today. For other video functions, we will open another report/branch. Feel free to submit your own additions to a bug report, and we'll be on our way!
        Hide
        Olle Johansson added a comment -

        Updated branch to trunk. Please test!

        Show
        Olle Johansson added a comment - Updated branch to trunk. Please test!
        Hide
        Olle Johansson added a comment -

        Seems like this code works and is ready to be implemented into trunk.

        Show
        Olle Johansson added a comment - Seems like this code works and is ready to be implemented into trunk.
        Hide
        Olle Johansson added a comment -

        Ready for svn, disclaimed, tested, confirmed.

        Show
        Olle Johansson added a comment - Ready for svn, disclaimed, tested, confirmed.
        Hide
        Olle Johansson added a comment -

        Committed to svn trunk rev 15148.

        Thanks to everyone involved in this work!

        Show
        Olle Johansson added a comment - Committed to svn trunk rev 15148. Thanks to everyone involved in this work!

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development