32803 Commits (0774d9f9aa663b42d9a40a15d017c817c12e3a5f)

 

All Branches   

Search

Author	SHA1	Message	Date
Ben Ford	168637cc0c	RTP/ICE: Send on first valid pair. ... When handling ICE negotiations, it's possible that there can be a delay between STUN binding requests which in turn will cause a delay in ICE completion, preventing media from flowing. It should be possible to send media when there is at least one valid pair, preventing this scenario from occurring. A change was added to PJPROJECT that adds an optional callback (on_valid_pair) that will be called when the first valid pair is found during ICE negotiation. Asterisk uses this to start the DTLS handshake, allowing media to flow. It will only be called once, either on the first valid pair, or when ICE negotiation is complete. ASTERISK-28716 Change-Id: Ia7b68c34f06d2a1d91c5ed51627b66fd0363d867	6 years ago
Sean Bright	8dcdce42a9	app_mixmonitor: Turn on synchronization by default ... The optional synchronization behavior created in 64906c4c9b is now the default for MixMonitor. * Add a new flag 'n' that allows for this behavior to be turned off * Add a notice when the 'S' option is used indicating that it is no longer necessary Change-Id: I158987c475cda4e1ff1256dd0daccdd99df568b4	6 years ago
George Joseph	1fc1336b2c	Merge "res_rtp_asterisk: bad audio (static) due to incomplete dtls/srtp setup"	6 years ago
Joshua Colp	12ba0682ed	Merge "stasis: Use format specifier for size_t."	6 years ago
Sean Bright	ddfb60ac2c	app_mixmonitor: Set MIXMONITOR_FILENAME to correct value when wav49 is used ... When opening a file for writing, Asterisk silently converts filenames ending with 'wav49' to 'WAV.' We aren't taking that in to account when setting the MIXMONITOR_FILENAME variable in MixMonitor. * If the user wants to write to a wav49 file, make sure that it is reflected properly in MIXMONITOR_FILENAME. * Add a note to the documentation describing this behavior. * Add a note in main/file.c indicating that app_mixmonitor needs to be changed if the logic in build_filename was changed. ASTERISK-24798 #close Reported by: xrobau Change-Id: I384691ce624eb55c80a125b9ca206d2d691c574c	6 years ago
Torrey Searle	bf4340f0ec	res_pjsip_sdp_rtp: implement hold state handling on moh_passthrough ... When moh_passthrough is used, asterisk is only generating invites of type sendonly and sendrecv instead of taking fully into account the on hold state of the local and remote parties ASTERISK-28738 #close Change-Id: Iaaad9fbc033cb14803d433b8a4071bc337047761	6 years ago
Joshua C. Colp	0f6ee98c3f	stasis: Use format specifier for size_t. ... Change-Id: Ic9b4afcc5398e7f46314419fc3c90433d818e35c	6 years ago
Kevin Harwell	3865b3fd6a	res_rtp_asterisk: bad audio (static) due to incomplete dtls/srtp setup ... There was a race condition between client initiated DTLS setup, and handling of server side ice completion that caused the underlying SSL object to get cleared during DTLS initialization. If this happened Asterisk would be left in a partial DTLS setup state. RTP packets were sent and received, but were not being encrypted and decrypted. This resulted in no audio, or static. Specifically, this occurred when '__rtp_recvfrom' was processing the handshake sequence from the client to the server, and then 'ast_rtp_on_ice_complete' gets called from another thread and clears the SSL object when calling the 'dtls_perform_setup' function. The timing had to be just right in the sense that from the external SSL library perspective SSL initialization completed (rtp recv), Asterisk clears/resets the SSL object (ice done), and then checks to see if SSL is intialized (rtp recv). Since it was cleared, Asterisk thinks it is not finished, thus not completing 'dtls_srtp_setup'. This patch removes calls to 'dtls_perform_setup', which clears the SSL object, in 'ast_rtp_on_ice_complete'. When ice completes, there is no reason to clear the underlying SSL object. If an ice candidate changes a full protocol level renegotiation occurs. Also, in the case of bundled ICE candidates are reused when a stream is added. So no real reason to have to clear, and reset in this instance. Also, this patch adds a bit of extra logging to aid in diagnosis of any future problems. ASTERISK-28742 #close Change-Id: I34c9e6bad5a39b087164646e2836e3e48fe6892f	6 years ago
Joshua Colp	519f21ecb2	Merge "res_pjsip_session: Fix off-nominal session refreshes."	6 years ago
Joshua Colp	4d671a2d14	Merge "res_musiconhold: Avoid spurious warning when 'format' is the empty string"	6 years ago
George Joseph	0d8275aa96	Merge "doc: Fix CHANGES entries to have .txt suffix and update READMEs"	6 years ago
Sean Bright	aeff1f2c53	res_musiconhold: Avoid spurious warning when 'format' is the empty string ... The change to res_config_odbc that allowed empty strings to be returned to realtime consumers¹ causes a warning to be emitted when loading MoH classes. So we need to treat an empty 'format' as if it was not specified to avoid the warning. ASTERISK-28735 #close Reported by: Ross Beer [1] https://gerrit.asterisk.org/c/asterisk/+/13722 Change-Id: I9a271d721e1a0973e80ebe7d75b46a0d8fa0e5a5	6 years ago
Sean Bright	1e037ebb97	func_odbc: Prevent snprintf() truncation warning ... For reasons that are not clear to me - this only appears for me when _not_ building in dev-mode. Change-Id: Ib45c54daaea8e0d571cb470cab1daaae2edba968	6 years ago
Joshua C. Colp	ac155decae	res_pjsip_session: Fix off-nominal session refreshes. ... Given a scenario where session refreshes occur close to each other while another is finishing it was possible for the session refreshes to occur out of order. It was also possible for session refreshes to be delayed for quite some time if a session refresh did not result in a topology change. For the out of order session refreshes the first session refresh would be queued due to a transaction in progress. This transaction would then finish. When finished a separate task to process the delayed requests queue would be queued for handling. A second refresh would be requested internally before this delayed request queued task was processed. As no transaction was in progress this session refresh would be immediately handled before the queued session refresh. The code will now check if any delayed requests exist before allowing a session refresh to immediately occur. If any exist then the session refresh is queued. For the delayed session refreshes if a session refresh did not result in a topology change the attempt would be immediately stopped and no other delayed requests would be processed. The code will now go through the entire delayed requests queue until a delayed request results in a request actually being sent. ASTERISK-28730 Change-Id: Ied640280133871f77d3f332be62265e754605088	6 years ago
George Joseph	a72caa041f	doc: Fix CHANGES entries to have .txt suffix and update READMEs ... Although the wiki page for the new CHANGES and UPGRADE scheme states that the files must have the ".txt" suffix, the READMEs didn't. Change-Id: I490306aa2cc24d6f014738e9ebbc78592efe0f05 (cherry picked from commit 7416703f04)	6 years ago
Joshua Colp	b2b6f28ac2	Merge "pjproject_bundled: Allow brackets in via parameters"	6 years ago
Friendly Automation	7c5a6efb44	Merge "install_prereq: Install aptitude non-interactively"	6 years ago
Joshua Colp	e5a53e50ba	Merge "chan_sip: Return 503 if we're out of RTP ports"	6 years ago
Sean Bright	9d9bde76a9	pjproject_bundled: Allow brackets in via parameters ... ASTERISK-26955 #close Reported by: Peter Sokolov Change-Id: Ib2803640905a77b65d0cee2d0ed2c7b310d470ac	6 years ago
Joshua Colp	67e4ec1a6c	Merge "chan_sip: Clarify in sample docs how directmediapermit/-acl should be used"	6 years ago
Joshua Colp	49809cb078	Merge "res_rtp_asterisk: Don't produce transport-cc if no packets."	6 years ago
Friendly Automation	dda7f986d0	Merge "res_config_odbc: Preserve empty strings returned by the database"	6 years ago
Friendly Automation	a41c971371	Merge "res_stasis_playback: Prevent media_index from going out of bounds"	6 years ago
Sylvain Afchain	0c02d0a450	install_prereq: Install aptitude non-interactively ... Currently aptitude is installed using interactive mode. This patch changes this to use the non-interactive mode as it can block automatic dependencies installation, ex: CI, Docker build. ASTERISK-28726 #close Change-Id: I271ee00d230513a6f044810351a32d83b2181133	6 years ago
Joshua C. Colp	1b53d329ac	res_rtp_asterisk: Don't produce transport-cc if no packets. ... The code assumed that when the transport-cc feedback function was called at least one packet will have been received. In practice this isn't always true, so now we just reschedule the sending and do nothing. Change-Id: Iabe7b358704da446fc3b0596b847bff8b8a0da6a	6 years ago
George Joseph	b76ab5e5c9	message.c: Add option to suppress the Message channel AMI and ARI events ... In order to reduce the amount of AMI and ARI events generated, the global "Message/ast_msg_queue" channel can be set to suppress it's normal channel housekeeping events such as "Newexten", "VarSet", etc. This can greatly reduce load on the manager and ARI applications when the Digium Phone Module for Asterisk is in use. To enable, set "hide_messaging_ami_events" in asterisk.conf to "yes" In Asterisk versions <18, the default is "no" preserving existing behavior. Beginning with Asterisk 18, the option will default to "yes". NOTE: This change does not affect UserEvents or the ARI TextMessageReceived events. * Added the "hide_messaging_ami_events" option to asterisk.conf. * Changed message.c to set the AST_CHAN_TP_INTERNAL property on the "Message/ast_msg_queue" channel if the option is set in asterisk.conf. This suppresses the reporting of the events. Change-Id: Ia2e3516d43f4e0df994fc6598565d6bba2d7018b	6 years ago
Joshua Colp	0c845063b3	Merge "res_pjsip_messaging: Allow Content-Type to be overridden"	6 years ago
Walter Doekes	43620cbf6c	chan_sip: Return 503 if we're out of RTP ports ... If you're for some reason out of RTP ports, chan_sip would previously responde to an INVITE with a 403, which will fail the call. Now, it returns a 503, allowing the device/proxy to retry the call on a different machine. ASTERISK-28718 Change-Id: I968dcf6c1e30ecddcce397dcda36db727c83ca90	6 years ago
Friendly Automation	d4b9529a58	Merge "res_stasis: trigger cleanup after update"	6 years ago
Friendly Automation	87ab83d03f	Merge "stasis/app: don't lock an app before a call to send"	6 years ago
Friendly Automation	499f41051e	Merge "res_pjsip_pubsub: Increment persistence data ref when recreating."	6 years ago
Sean Bright	eb9252ea27	res_config_odbc: Preserve empty strings returned by the database ... When res_config_odbc (and perhaps other realtime backends) reads a SQL NULL from the database, it coalesces the value to the empty string which prevents it from being returned to the realtime core. However, if it instead reads the empty string from the database, it needs a way to encode that fact without having the value omitted entirely. It does this by changing the value to a string with a single space. The realtime code in main/config.c recognizes this special case and _turns the string back into the empty string_ before passing it to realtime API consumers. For all of this to work, we need to ensure that we actually pass the single-space-string back to the realtime core, which is currently failing because we are trimming the value before checking its content. So instead we now special case the single-space-string case so that empty values are returned properly. ASTERISK-28719 #close Reported by: EDV O-TON Change-Id: I673ed8c31ad037aa224e80c78c7a1dc4e4a4e3de	6 years ago
Sean Bright	31dc904380	res_stasis_playback: Prevent media_index from going out of bounds ... Incrementing stasis_app_playback.media_index directly in our playback loop means that when we reach the end of our playlist the index into the vector will be outside of the bounds of the vector. Instead use a temporary variable and only assign when we're sure that we are in bounds. ASTERISK-28713 #close Reported by: Sébastien Duthil Change-Id: Ib53f7f156097e0607eb5871d9d78d246ed274928	6 years ago
Friendly Automation	e2119e8968	Merge "res_pjsip_messaging: Ensure MESSAGE_SEND_STATUS is set properly"	6 years ago
Joshua C. Colp	a1f0c833ab	res_pjsip_pubsub: Increment persistence data ref when recreating. ... Each subscription needs to have a reference to the persisted data for it, as well as the main JSON contained within the tree. When recreating a subscription this did not occur and they both shared the same reference. ASTERISK-28714 Change-Id: I706abd49ea182ea367a4ac3feca2706460ae9f4a	6 years ago
Sean Bright	03d24ca4c1	res_pjsip_messaging: Allow Content-Type to be overridden ... ASTERISK-26082 #close Reported by: Alex Change-Id: I6549e90932016349bc72b0f053432dc25286f4fb	6 years ago
Walter Doekes	113d05e504	chan_sip: Clarify in sample docs how directmediapermit/-acl should be used ... It said "restrict [...] which peers should be able to pass [audio] to each other". However, these settings are not global (for which you would expect signaling IPs to be checked). These settings are available per peer only, and the IPs being checked, are the RTP IPs. Change-Id: I2a6c6cd7c2f5f30d1df4844e3e0308a077021660	6 years ago
Friendly Automation	f29ddd8925	Merge "chan_sip: Always process updated SDP on media source change"	6 years ago
Kevin Harwell	cce2b0da95	stasis/app: don't lock an app before a call to send ... Calling 'app_send' eventually calls the app's message handler. It's possible for a handler to obtain a lock on another object, and then need/want to lock the app object. If the caller of 'app_send' locks the app object prior to calling then there's a potential for a deadlock, if another thread calls 'app_send' without locking. This patch makes it so 'app_send' is not called with the app object locked in the section of code doing such. ASTERISK-28423 #close Change-Id: I6767c6d0933c7db1b984018966eefca4c0638a27	6 years ago
Kevin Harwell	4206830a52	res_stasis: trigger cleanup after update ... The cleanup code in stasis shuts down applications if they are in a deactivated state, and no longer have explicit subscriptions. When registering an app the cleanup code was running before calling 'update'. When it should be executed after 'update' since a call to register may re-activate the app. We don't want it to shutdown before the 'update' otherwise the app won't be re-activated, or registered. This patch makes it so the cleanup code is executed post 'update'. ASTERISK-28679 #close Change-Id: I8f2c0b17e33bb8128441567b97fd4c7bf74a327b	6 years ago
Sean Bright	b1ca2c5d71	res_pjsip_messaging: Ensure MESSAGE_SEND_STATUS is set properly ... We need to wait for the message sending callback to finish to know if we succeeded or failed. ASTERISK-25421 #close Reported by: Dmitriy Serov Change-Id: I22b954398821d2caf4c6fe58f0607c8cfa378059	6 years ago
Walter Doekes	711a3fed56	chan_sip: Always process updated SDP on media source change ... Fixes no-audio issues when the media source is changed and strictrtp is enabled (default). If the peer media source changes, the SDP session version also changes. If it is lower than the one we had stored, chan_sip would ignore it. This changeset keeps track of the remote media origin identifier, comparing that as well. If it changes, the session version needn't be higher for us to accept the SDP. Common scenario where this would've caused problems: a separate media gateway that informs the caller about premium rates before handing off the call to the final destination. (An alternative fix would be to set ignoresdpversion=yes on the peer.) ASTERISK-28686 Change-Id: I88fdbc5aeb777b583e7738c084254c482a7776ee	6 years ago
Sean Bright	313189aae2	chan_pjsip: Ignore RTP that we haven't negotiated ... If chan_pjsip receives an RTP packet whose payload differs from the channel's native format, and asymmetric_rtp_codec is disabled (the default), Asterisk will switch the channel's native format to match that of the incoming packet without regard to the negotiated payloads. We now check that the received frame is in a format we have negotiated before switching payloads which results in these packets being dropped instead of causing the session to terminate. ASTERISK-28139 #close Reported by: Paul Brooks Change-Id: Icc3b85cee1772026cee5dc1b68459bf9431c14a3	6 years ago
George Joseph	9688381f05	Merge "http: Add ability to disable /httpstatus URI"	6 years ago
Joshua Colp	b88791d88c	Merge "cdr.c: Set event time on party b when leaving a parking bridge"	6 years ago
Friendly Automation	95c6fbeae0	Merge "app_voicemail: Remove MessageExists and MESSAGE_EXISTS()"	6 years ago
George Joseph	6818c3d1d2	cdr.c: Set event time on party b when leaving a parking bridge ... When Alice calls Bob and Bob does a blind transfer to Charlie, Bob's bridge leave event generates a finalize on both the party_a and party_b CDRs but while the party_a CDR has the correct end time set from the event time, party_b's leg did not. This caused that CDR's end time to be equal to the answered time and resulted in a billsec of 0. * We now pass the bridge leave message event time to cdr_object_party_b_left_bridge_cb() and set it on that CDR before calling cdr_object_finalize() on it. NOTE: This issue affected transfers using chan_sip most of the time but also occasionally affected chan_pjsip probably due to message timing. ASTERISK-28677 Reported by: Maciej Michno Change-Id: I790720f1e7326f9b8ce8293028743b0ef0fb2cca	6 years ago
Sean Bright	0dce6f746b	http: Add ability to disable /httpstatus URI ... Add a new configuration option 'enable_status' which allows the /httpstatus URI handler to be administratively disabled. We also no longer unconditionally register the /static and /httpstatus URI handlers, but instead do it based upon configuration. Behavior change: If enable_static was turned off, the URI handler was still installed but returned a 403 when it was accessed. Because we now register/unregister the URI handlers as appropriate, if the /static URI is disabled we will return a 404 instead. Additionally: * Change 'enablestatic' to 'enable_static' but keep the former for backwards compatibility. * Improve some internal variable names ASTERISK-28710 #close Change-Id: I647510f796473793b1d3ce1beb32659813be69e1	6 years ago
Joshua Colp	b073b4318a	Merge "func_odbc.conf.sample: Add example lookup"	6 years ago
Friendly Automation	9c1462f0e4	Merge "res_statsd: Document that res_statsd does nothing on its own"	6 years ago