Skip to content
Snippets Groups Projects
  1. Mar 29, 2017
  2. Mar 28, 2017
  3. Mar 27, 2017
    • Sean Bright's avatar
      res_musiconhold: Document the 'format' option · fd204d5c
      Sean Bright authored
      ASTERISK-26086 #close
      Reported by: Jens Bürger
      
      Change-Id: I6aab666c0bf01fd0c64d7a5bcb22fa7f5d41335e
      fd204d5c
    • Sean Bright's avatar
      res_musiconhold: Don't chdir() when scanning MoH files · d22c6789
      Sean Bright authored
      There doesn't appear to be any reason that we are chdir'ing in
      moh_scan_files, and in the event of an Asterisk crash, the core files
      may not get written because we have changed into a read-only directory.
      
      ASTERISK-23996 #close
      Reported by: Walter Doekes
      
      Change-Id: Iac806dce01b3335963fbd62d4b4da9a65c614354
      d22c6789
  4. Mar 25, 2017
  5. Mar 24, 2017
  6. Mar 23, 2017
    • Kevin Harwell's avatar
      AMI: Updated version · d2f2cdf4
      Kevin Harwell authored
      Updated the AMI version for the following reason (see CHANGES for more details):
      
      The 'PJSIPShowEndpoint' command's response event of 'IdentifyDetail' now
      contains a new optional parameter, 'MatchHeader'.
      
      Change-Id: Ie206913ef1dcfa6a2ebe3282da2387e52d6f05b9
      d2f2cdf4
    • Kevin Harwell's avatar
      pjproject_bundled: raise timeout value used when downloading · 12dde3b5
      Kevin Harwell authored
      After configuring Asterisk with '--with-pjproject-bundled' the configure/build
      process attempts to download pjproject from its download site. Currently, a
      timeout of 10 seconds is used that will stop the download process if pjproject
      has not been fully downloaded in that time. For some systems this was not enough
      time and the process was timing out too early.
      
      This patch raises the download timeout value to '60'. Also, this patch fixes
      another bug where the DOWNLOAD_TIMEOUT variable was not being properly exported
      due to a naming error. DOWNLOAD_MAX_TIMEOUT is now properly renamed to
      DOWNLOAD_TIMEOUT.
      
      ASTERISK-26814 #close
      
      Change-Id: Ia56e4e8a3d39db76bc8a1852b2cf07ec10b39842
      12dde3b5
    • Sean Bright's avatar
      res_xmpp: Correct implementation of JABBER_STATUS & JabberStatus · 98a88e9f
      Sean Bright authored
      The documentation for JABBER_STATUS (and the deprecated JabberStatus
      app) indicate that a return value of 7 indicates that the specified
      buddy was not in the roster. It also indicates that you can specify a
      "bare" JID (one without a resource). Unfortunately the actual behavior
      does not match the documented behavior.
      
      Assuming that our roster includes the buddy online and available
      "valid@example.org/Valid" and does *not* include the buddy
      "invalid@example.org", the JABBER_STATUS() function returns the
      following before this patch:
      
      +------------------------------+------------+--------------------------+
      | Buddy                        | Status     | Result                   |
      +------------------------------+------------+--------------------------+
      | valid@example.org            |  Online    |  7 (Not in roster)       |
      | valid@example.org/Valid      |  Online    |  1 (Online)              |
      | valid@example.org/Invalid    |  N/A       |  7 (Not in roster)       |
      | invalid@example.org          |  N/A       |  Error logged, no return |
      | invalid@example.org/Valid    |  N/A       |  Error logged, no return |
      +------------------------------+------------+--------------------------+
      
      And after this patch:
      
      +------------------------------+------------+--------------------------+
      | Buddy                        | Status     | Result                   |
      +------------------------------+------------+--------------------------+
      | valid@example.org            |  Online    |  1 (Online)              |
      | valid@example.org/Valid      |  Online    |  1 (Online)              |
      | valid@example.org/Invalid    |  N/A       |  6 (Offline)             |
      | invalid@example.org          |  N/A       |  7 (Not in roster)       |
      | invalid@example.org/Valid    |  N/A       |  7 (Not in roster)       |
      +------------------------------+------------+--------------------------+
      
      This brings the behavior in line with the documentation.
      
      ASTERISK-23510 #close
      Reported by: Anthony Critelli
      
      Change-Id: I9c3241035363ef4a6bdc21fabfd8ffcd9ec657bf
      98a88e9f
    • Sean Bright's avatar
      res_xmpp: Try to provide useful errors messages from OpenSSL · be94105d
      Sean Bright authored
      If any errors occur during the TLS connection setup, we currently dump a
      fairly generic error message. So instead we try to pull in something
      useful from OpenSSL to report instead.
      
      ASTERISK-24712
      Reported by: Matthias Urlichs
      
      Change-Id: I288500991a9681f447d92913b11fedaf426087f4
      be94105d
    • Sean Bright's avatar
      res_xmpp: Fix ref counting issue · ee81ee1f
      Sean Bright authored
      The only remaining reference to the endpoint is in the endpoints
      container, and because it is unlinked in ast_endpoint_shutdown, we don't
      have to explicitly cleanup the endpoint ourselves.
      
      Change-Id: I912a2692e52d3e2ed445b32d8ae3f9004bc2f2e8
      ee81ee1f
    • Sean Bright's avatar
      res_xmpp: Correctly check return value of SSL_connect · 94939814
      Sean Bright authored
      SSL_connect returns non-zero for both success and some error conditions
      so simply negating is inadequate.
      
      Change-Id: Ifbf882896e598703b6c615407fa456d3199f95b1
      94939814
    • Sean Bright's avatar
      res_xmpp: Don't crash when trying to send a message without a connection · 7657c279
      Sean Bright authored
      If we never establish a connection to our Jabber server, iksemel never sets up
      its internal transport pointer, so attempting to send a message dereferences a
      NULL pointer and causes a crash.
      
      ASTERISK-21855 #close
      Reported by: Jeremy Kister
      
      Change-Id: I204a568894e4a53ab929783ecc594a000f04d79c
      7657c279
    • Sean Bright's avatar
      res_xmpp: Include client name in connection related error messages · 0136ec12
      Sean Bright authored
      ASTERISK-25622 #close
      Reported by: Sean Darcy
      
      Change-Id: I8472cb7bfb58d411a3cfbd482da98cae2d94d1e9
      0136ec12
  7. Mar 22, 2017
  8. Mar 21, 2017
Loading