4448 Commits (bbff402be66e3c2dc28ec6c4e82f5e088fc7ebd3)

Author	SHA1	Message	Date
Jenkins2	1d9ef9f5f3	Merge "endpoint identifiers: Some code cleanup." into 15	8 years ago
Richard Mudgett	53afe4660a	res_pjsip/config_domain_aliases.c: Add check for missing domain. ... What is the point of defining an alias and not saying what is being aliased? Change-Id: I98a892016ed61dcf5efeb6619fd748925103f0be	8 years ago
Jenkins2	65ad576017	Merge "res_sorcery_realtime.c: Fix ref leak if object failed to apply." into 15	8 years ago
Jenkins2	fc55428a42	Merge "res_pjsip.c: Fix documentation typos." into 15	8 years ago
Richard Mudgett	c3916cbc0e	endpoint identifiers: Some code cleanup. ... res_pjsip_endpoint_identifier_user.c: * Fix copy/paste error in find_endpoint(). We were using a constant "anonymous" string instead of the passed in endpoint_name when checking the transport domain for an endpoint match. * Eliminate RAII_VAR in find_endpoint(). * Remove always true check in find_transport_state_in_use(). * Remove useless CMD_STOP in find_transport_state_in_use(). res_pjsip_endpoint_identifier_anonymous.c: * Eliminate RAII_VAR in anonymous_identify(). * Remove always true check in find_transport_state_in_use(). * Remove useless CMD_STOP in find_transport_state_in_use(). Change-Id: I86924c31db5bd225ca0c1219c761b668c6f91189	8 years ago
Richard Mudgett	3cc4cb9c68	res_pjsip.c: Fix documentation typos. ... Change-Id: I82ae0b92bfa2ece84a5c684efd9eefdc83ebd068	8 years ago
Richard Mudgett	cda2c914b6	res_sorcery_realtime.c: Fix ref leak if object failed to apply. ... Change-Id: I3c7106ff77009754725cee790eadf5da44154ab6	8 years ago
Joshua Elson	296711a446	res_pjsip_mwi.c: Fix null pointer crash ... ASTERISK-27652 #close Change-Id: I78a0d38bfd8d0d82830f3d53da04872d6b67284d	8 years ago
George Joseph	90e5bae6ca	Merge "res_pjsip_pubsub: Prune subs with reliable transports at startup" into 15	8 years ago
Jenkins2	29be9661a3	Merge "res_pjsip_registrar_expire: Mark module deprecated." into 15	8 years ago
Jenkins2	def9790012	Merge "res_pjsip_session: Prevent crash during shutdown." into 15	8 years ago
Corey Farrell	2a05a05038	res_pjsip_registrar_expire: Mark module deprecated. ... The functionality of this module was already moved to res_pjsip_registrar, mark it deprecated and add message to CHANGES. Change-Id: I90c7d52c7e15e85fde3389d5eaccb05b97848813	8 years ago
Richard Mudgett	5dce58ea94	res_pjsip_registrar.c: Fix compiler error. ... Need to include signal.h to define pthread_kill() and SIGURG. Change-Id: I10ae3aa4bf8e7386ac29ade78c0f2caed8e674fa	8 years ago
Corey Farrell	0b97170588	res_pjsip_session: Prevent crash during shutdown. ... pjproject does not have a function to reverse pjsip_inv_usage_init. This means we need to ignore any calls to the functions once shutdown is final. ASTERISK-27571 #close Change-Id: Ia550fcba563e2328f03162d79fb185f16b7c9b9d	8 years ago
George Joseph	af2fedb6ba	res_pjsip_pubsub: Prune subs with reliable transports at startup ... In an earlier release, inbound registrations on a reliable transport were pruned on Asterisk restart since the TCP connection would have been torn down and become unusable when Asterisk stopped. This same process is now also applied to inbound subscriptions. Also fixed issues in res_pjsip_registrar where it wasn't handling the monitoring correctly when multiple registrations came in over the same transport. To accomplish this, the pjsip_transport_event feature needed to be refactored to allow multiple monitors (multiple subcriptions or registrations from the same endpoint) to exist on the same transport. Since this changed the API, any external modules that may have used the transport monitor feature (highly unlikey) will need to be changed. ASTERISK-27612 Reported by: Ross Beer Change-Id: Iee87cf4eb9b7b2b93d5739a72af52d6ca8fbbe36	8 years ago
George Joseph	7bc91d06df	res_pjsip_registrar_expire: Refactor into res_pjsip_register ... res_pjsip_registrar_expire remains as an empty module for now. Change-Id: Ib93698938bae548d2199cb542f3692d1a171239f	8 years ago
Sean Bright	0361a073fe	res_pjsip: Document tlsv1_1 and tlsv1_2 methods ... Change-Id: I67ed9039bf3f132fb20ee7a750e0aef0f704d7d3	8 years ago
Jenkins2	c167808c9f	Merge "res_pjsip: Split type=identify to IP address and SIP header matching priorities" into 15	8 years ago
Jenkins2	0153081f9b	Merge "res_stasis_recording: Allow symbolic links in configured recordings dir." into 15	8 years ago
Jenkins2	f88ff34c47	Merge "config_transport: Enable TCP_NODELAY on TLS transports" into 15	8 years ago
George Joseph	176a09e3f3	config_transport: Enable TCP_NODELAY on TLS transports ... We did this for TCP transports already but I'm not sure why we didn't do it for TLS transports. ASTERISK_27474 #not_final_fix Change-Id: I5b1ef4b882f7b859e718236686b7898751dbb262	8 years ago
Corey Farrell	deebabfdb1	res_stasis_recording: Allow symbolic links in configured recordings dir. ... If any component of ast_config_AST_RECORDING_DIR is a symbolic link we would incorrectly assume the ARI user was trying to escape the recording path. Create additional check to check the recording directory's realpath, only deny access if both do not match. This is needed by the testsuite when run by 'run-local'. Change-Id: I9145e841865edadcb5f75cead3471ad06bbb56c0	8 years ago
Alexander Traud	45d2c913f4	res_config_pgsql: Avoid typecasting an int to unsigned char. ... clang 5.0 warned about this. ASTERISK-27576 Change-Id: If41f400a51973c06cdb9b75462e535b616bfe385	8 years ago
Richard Mudgett	821b718a96	res_pjsip: Split type=identify to IP address and SIP header matching priorities ... The type=identify endpoint identification method can match by IP address and by SIP header. However, the SIP header matching has limited usefulness because you cannot specify the SIP header matching priority relative to the IP address matching. All the matching happens at the same priority and the order of evaluating the identify sections is indeterminate. e.g., If you had two type=identify sections where one matches by IP address for endpoint alice and the other matches by SIP header for endpoint bob then you couldn't predict which endpoint is matched when a request comes in that matches both. * Extract the SIP header matching criteria into its own "header" endpoint identification method so the user can specify the relative priority of the SIP header and the IP address matching criteria in the global endpoint_identifier_order option. The "ip" endpoint identification method now only matches by IP address. ASTERISK-27491 Change-Id: I9df142a575b7e1e3471b7cda5d3ea156cef08095	8 years ago
Jenkins2	ac8a549b9e	Merge "res_pjsip_endpoint_identifier_ip.c: Remove unnecessary requirement." into 15	8 years ago
Jenkins2	6fa4f69d50	Merge "res_pjsip.c: Update the endpoint identification documentation." into 15	8 years ago
Jenkins2	9511f2c3af	Merge "res_pjsip.c: Fix ident_to_str() and refactor ident_handler()." into 15	8 years ago
Joshua Colp	b4d2d91d43	Merge "res_pjsip_endpoint_identifier_ip.c: Allow multiple IdentifyDetail AMI events." into 15	8 years ago
Joshua Colp	0bcd02b2e0	Merge "res_stasis: Reduce RAII_VAR usage." into 15	8 years ago
Jenkins2	411a7f12dc	Merge "res_pjsip_session: Always bundle streams if WebRTC is enabled." into 15	8 years ago
Richard Mudgett	6d9482267b	res_pjsip.c: Update the endpoint identification documentation. ... * Endpoint identify_by documentation. * IP/Header endpoint identifier documentation. Change-Id: Id92f00b495acca7be945daf749d2abd7f76a0b5a	8 years ago
Richard Mudgett	74ba8398fe	res_pjsip_endpoint_identifier_ip.c: Remove unnecessary requirement. ... The requirement that "ip" must be in the endpoint identify_by list to allow the type=identify method to identify the endpoint is not necessary. The "ip" identifier method can match one and only one endpoint. To even work, the "ip" identifier method configuration must explicitly specify the identified endpoint. Therefore, why bother configuring the type=identify identifier in the first place? The requirement only adds the potential for configuration errors for no benefit. Even worse, those configuration errors cannot be detected when the configuration loads. The requirement was introduced with the ASTERISK_27206 patch. * Remove the code change that enforces the requiremnt. Listing the "ip" method in the identify_by value is simply documentation. Change-Id: Ia057f92a33fb5d9f51dc5d5692e3d5ee1a6f2c11	8 years ago
Richard Mudgett	3a4e949a0e	res_pjsip.c: Fix ident_to_str() and refactor ident_handler(). ... * Extracted sip_endpoint_identifier_type2str() and sip_endpoint_identifier_str2type() to simplify the calling functions. * Fixed pjsip_configuration.c:ident_to_str() building the endpoint's identify_by value string. Change-Id: Ide876768a8d5d828b12052e2a75008b0563fc509	8 years ago
Richard Mudgett	2b24d941b9	res_pjsip_endpoint_identifier_ip.c: Allow multiple IdentifyDetail AMI events. ... The AMI PJSIPShowEndpoint action could only list one IdentifyDetail AMI event per endpoint. However, there is no reason that multiple type=identify sections cannot identify the same endpoint. * Reworked format_ami_endpoint_identify() to generate as many IdentifyDetail AMI events as there are matching identifiers. Change-Id: Ie146792aef72d78e05416ab5b27bc552a30399db	8 years ago
Jenkins2	eb928b392e	Merge "res_pjsip.c: Fix endpoint identifier registration name search." into 15	8 years ago
Jenkins2	475f337272	Merge "res_stasis: Fix dial bridge unload." into 15	8 years ago
Jenkins2	37e57a623a	Merge "res_pjsip_endpoint_identifier_ip.c: Fix apply identify validation." into 15	8 years ago
Corey Farrell	b18c4d2de6	res_stasis: Reduce RAII_VAR usage. ... In addition to being a micro-optimization (RAII_VAR has overhead), this change improves output of REF_DEBUG. Unfortunately when RAII_VAR calls ao2_cleanup it does so from a generated _dtor_varname function. For example this caused _dtor_app to release a reference instead of __stasis_app_unregister. Change-Id: I4ce67120583a446babf9adeec678b71d37fcd9e5	8 years ago
Jenkins2	32817bf389	Merge "res_stasis: Fix app_is_subscribed_bridge_id." into 15	8 years ago
Joshua Colp	5b2a65ea8b	res_pjsip_session: Always bundle streams if WebRTC is enabled. ... Some WebRTC clients can't handle renegotiation with the addition of streams that include an offer to bundle. They instead expect the newly added streams to already be bundled. This change does such a thing if WebRTC support is enabled on an endpoint. ASTERISK-27566 Change-Id: I7fe9b7ac35a2798627d9c2c8369129f407af6461	8 years ago
Jenkins2	fa93645a85	Merge "General: Avoid implicit conversion to char when changes value to negative." into 15	8 years ago
Corey Farrell	9876bb993c	res_stasis: Fix dial bridge unload. ... If the dial bridge has been created it must be released by calling ast_bridge_destroy, simply releasing the ao2 reference is not enough. Also move stasis_app_control_shutdown earlier in unload to ensure the bridge cannot be created or grabbed after the app_bridges container is released. Change-Id: I372302de94ca63876069e2585a049c5060e5e767	8 years ago
Corey Farrell	db72046e1f	res_stasis: Fix app_is_subscribed_bridge_id. ... Instead of searching for bridge_id provided in an argument this function always searched for BRIDGE_ALL first. Rewrite this function to work like the similar functions for channel and endpoint functions. Change-Id: Ib5caca69e11727c5c8a7284a1d00621f40f1e60a	8 years ago
Alexander Traud	7b20a86c7d	General: Silence modules on (un)load. ... Some (normally optional) modules created notices, warnings, and even errors in normal situations like (un)load. This cluttered the command-line interface (CLI) on start and while stopping gracefully. However, when an user went for the script './contrib/scripts/install_prereq', those modules get compiled-in because their prerequisites were met at compile time. Furthermore, because of ASTERISK_27475, the former talkative module 'res_curl' is built as side-effect. ASTERISK-27553 Change-Id: I9f105f46d72553994e820679bfde3478a551b281	8 years ago
Alexander Traud	c33da7be98	General: Avoid implicit conversion to char when changes value to negative. ... clang 5.0 warned about this. ASTERISK-27557 Change-Id: I7cceaa88e147cbdf81a3a7beec5c1c20210fa41e	8 years ago
Richard Mudgett	ca63534faf	res_pjsip_endpoint_identifier_ip.c: Fix apply identify validation. ... The ip_identify_apply() did not validate the configuration for simple static configuration errors or deal well with address resolution errors. * Added missing configuration validation checks. * Fixed address resolution error handling. * Demoted an error message to a warning since it does not fail applying the identify object configuration. Change-Id: I8b519607263fe88e8ce964f526a45359fd362b6e	8 years ago
Richard Mudgett	cb41d16827	res_pjsip.c: Fix endpoint identifier registration name search. ... If an endpoint identifier name in the endpoint_identifier_order list is a prefix to the identifier we are registering, we could install it in the wrong position of the list. Assuming endpoint_identifier_order=username,ip,anonymous then registering the "ip_only" identifier would put the identifier in the wrong position of the priority list. * Fix incorrect strncmp() string prefix matching. Change-Id: Ib8819ec4b811da8a27419fd93528c54d34f01484	8 years ago
Corey Farrell	1341210ee2	res_pjsip_history: Add missing unlock to CLI command. ... Change-Id: I872060a30543776a176a316309602d924a23eb29	8 years ago
Kevin Harwell	02452ef4d1	res_pjsip_session: Check if sequence header is missing ... The pjsip_msg_find_hdr function can return NULL. This patch adds a check when searching for the sequence header to make sure a NULL pointer is never de-referenced. Change-Id: I19af23aeeded65be016be92360e8cb7ffe51fad2	8 years ago
Sean Bright	9a332027d4	ice: Increase foundation buffer size ... Per RFC 5245, the foundation specified with an ICE candidate can be up to 32 characters but we are only allowing for 31. ASTERISK-27498 #close Reported by: Michele Prà Change-Id: I05ce7a5952721a76a2b4c90366168022558dc7cf	8 years ago