<p>Mark Murawski has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14405">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pjsip:  Add endpoint configuration -- log_subscription_error<br><br>Turn off logging for any kind of PJSIP subscription error<br>Example: Endpoint '1000' state subscription failed: Extension '1010' does not exist in context....<br><br>Patches:<br>  master-6b2d9451741cadb080637190af65d40f692d67ed-pjsip-log_subscription_error.patch  submitted by Mark Murawski (license 1000250)<br><br>ASTERISK-28881 #close<br><br>Change-Id: I5dff21c3c8ac3a3e3aefbd89053581fb90bc7018<br>---<br>M include/asterisk/res_pjsip.h<br>M res/res_pjsip/pjsip_configuration.c<br>M res/res_pjsip_exten_state.c<br>3 files changed, 8 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/05/14405/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/asterisk/res_pjsip.h b/include/asterisk/res_pjsip.h</span><br><span>index fd80581..846f929 100644</span><br><span>--- a/include/asterisk/res_pjsip.h</span><br><span>+++ b/include/asterisk/res_pjsip.h</span><br><span>@@ -598,6 +598,8 @@</span><br><span>        struct ast_sip_mwi_configuration mwi;</span><br><span>        /*! Context for SUBSCRIBE requests */</span><br><span>        char context[AST_MAX_CONTEXT];</span><br><span style="color: hsl(120, 100%, 40%);">+        /*! Whether or not to log failed notify (Example: Endpoint '1000' state subscription failed: Extension '1010' does not exist in context.....) */</span><br><span style="color: hsl(120, 100%, 40%);">+      unsigned int log_subscription_error;</span><br><span> };</span><br><span> </span><br><span> /*!</span><br><span>diff --git a/res/res_pjsip/pjsip_configuration.c b/res/res_pjsip/pjsip_configuration.c</span><br><span>index f95ee9e..088c950 100644</span><br><span>--- a/res/res_pjsip/pjsip_configuration.c</span><br><span>+++ b/res/res_pjsip/pjsip_configuration.c</span><br><span>@@ -2025,6 +2025,7 @@</span><br><span>               call_offer_pref_handler, incoming_call_offer_pref_to_str, NULL, 0, 0);</span><br><span>       ast_sorcery_object_field_register_custom(sip_sorcery, "endpoint", "outgoing_call_offer_pref", "remote",</span><br><span>                call_offer_pref_handler, outgoing_call_offer_pref_to_str, NULL, 0, 0);</span><br><span style="color: hsl(120, 100%, 40%);">+       ast_sorcery_object_field_register(sip_sorcery, "endpoint", "log_subscription_error", "yes", OPT_BOOL_T, 1, FLDSET(struct ast_sip_endpoint, subscription.log_subscription_error));</span><br><span> </span><br><span>   if (ast_sip_initialize_sorcery_transport()) {</span><br><span>                ast_log(LOG_ERROR, "Failed to register SIP transport support with sorcery\n");</span><br><span>diff --git a/res/res_pjsip_exten_state.c b/res/res_pjsip_exten_state.c</span><br><span>index df9a35f..9b72d69 100644</span><br><span>--- a/res/res_pjsip_exten_state.c</span><br><span>+++ b/res/res_pjsip_exten_state.c</span><br><span>@@ -415,9 +415,11 @@</span><br><span>     const char *context = S_OR(endpoint->subscription.context, endpoint->context);</span><br><span> </span><br><span>     if (!ast_exists_extension(NULL, context, resource, PRIORITY_HINT, NULL)) {</span><br><span style="color: hsl(0, 100%, 40%);">-              ast_log(LOG_NOTICE, "Endpoint '%s' state subscription failed: "</span><br><span style="color: hsl(0, 100%, 40%);">-                       "Extension '%s' does not exist in context '%s' or has no associated hint\n",</span><br><span style="color: hsl(0, 100%, 40%);">-                  ast_sorcery_object_get_id(endpoint), resource, context);</span><br><span style="color: hsl(120, 100%, 40%);">+              if (endpoint->subscription.log_subscription_error) {</span><br><span style="color: hsl(120, 100%, 40%);">+                       ast_log(LOG_NOTICE, "Endpoint '%s' state subscription failed: "</span><br><span style="color: hsl(120, 100%, 40%);">+                             "Extension '%s' does not exist in context '%s' or has no associated hint\n",</span><br><span style="color: hsl(120, 100%, 40%);">+                                ast_sorcery_object_get_id(endpoint), resource, context);</span><br><span style="color: hsl(120, 100%, 40%);">+              }</span><br><span>            return 404;</span><br><span>  }</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14405">change 14405</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/14405"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I5dff21c3c8ac3a3e3aefbd89053581fb90bc7018 </div>
<div style="display:none"> Gerrit-Change-Number: 14405 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>