res/res_pjsip_exten_state: Fix confusing NOTICE message
authorMatt Jordan <mjordan@digium.com>
Fri, 22 May 2015 17:18:31 +0000 (12:18 -0500)
committerJoshua Colp <jcolp@digium.com>
Fri, 22 May 2015 17:23:52 +0000 (12:23 -0500)
When a SUBSCRIBE request is made to a dialplan hint that doesn't exist,
the current NOTICE message informing users of this swaps the context and
extension parameters. This can cause a bit of confusion.

Thanks to CptBurger in #asterisk for helping to point this out.

Change-Id: Ie584d1a58ae217385c87a450ca25b55ca0e36e43

res/res_pjsip_exten_state.c

index 5e8a017..a8a11bc 100644 (file)
@@ -354,7 +354,7 @@ static int new_subscribe(struct ast_sip_endpoint *endpoint,
 {
        if (!ast_exists_extension(NULL, endpoint->context, resource, PRIORITY_HINT, NULL)) {
                ast_log(LOG_NOTICE, "Extension state subscription failed: Extension %s does not exist in context '%s' or has no associated hint\n",
-                       endpoint->context, resource);
+                       resource, endpoint->context);
                return 404;
        }