diff options
author | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-11-02 20:51:53 +0000 |
---|---|---|
committer | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-11-02 20:51:53 +0000 |
commit | 5590a16c770cba2640bd52ee9aca66333e9c8644 (patch) | |
tree | 89f2c0fcc38d72bc17038e56d0c5d0cb862ad112 | |
parent | 077383a065e50e9709688b8d8418000d2291040d (diff) |
Merged revisions 88366 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r88366 | file | 2007-11-02 17:49:45 -0300 (Fri, 02 Nov 2007) | 4 lines
Make subscribecontext behave as advertised. It will now look for the presence of a hint in the given context (be it subscribecontext or context).
(closes issue #10702)
Reported by: slavon
........
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@88367 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_sip.c | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 61e36051d..d895a3689 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -9560,17 +9560,27 @@ static int get_destination(struct sip_pvt *p, struct sip_request *oreq) ast_string_field_set(p, context, domain_context); } + /* If the request coming in is a subscription and subscribecontext has been specified use it */ + if (req->method == SIP_SUBSCRIBE && !ast_strlen_zero(p->subscribecontext)) + ast_string_field_set(p, context, p->subscribecontext); + if (sip_debug_test_pvt(p)) ast_verbose("Looking for %s in %s (domain %s)\n", uri, p->context, p->domain); - /* Check the dialplan for the username part of the request URI, - the domain will be stored in the SIPDOMAIN variable - Return 0 if we have a matching extension */ - if (ast_exists_extension(NULL, p->context, uri, 1, from) || - !strcmp(uri, ast_pickup_ext())) { - if (!oreq) - ast_string_field_set(p, exten, uri); - return 0; + /* If this is a subscription we actually just need to see if a hint exists for the extension */ + if (req->method == SIP_SUBSCRIBE) { + char hint[AST_MAX_EXTENSION]; + return (ast_get_hint(hint, sizeof(hint), NULL, 0, NULL, p->context, p->exten) ? 0 : -1); + } else { + /* Check the dialplan for the username part of the request URI, + the domain will be stored in the SIPDOMAIN variable + Return 0 if we have a matching extension */ + if (ast_exists_extension(NULL, p->context, uri, 1, from) || + !strcmp(uri, ast_pickup_ext())) { + if (!oreq) + ast_string_field_set(p, exten, uri); + return 0; + } } /* Return 1 for pickup extension or overlap dialling support (if we support it) */ @@ -15891,16 +15901,6 @@ static int handle_request_subscribe(struct sip_pvt *p, struct sip_request *req, /* Get destination right away */ gotdest = get_destination(p, NULL); - /* Initialize the context if it hasn't been already; - note this is done _after_ handling any domain lookups, - because the context specified there is for calls, not - subscriptions - */ - if (!ast_strlen_zero(p->subscribecontext)) - ast_string_field_set(p, context, p->subscribecontext); - else if (ast_strlen_zero(p->context)) - ast_string_field_set(p, context, default_context); - /* Get full contact header - this needs to be used as a request URI in NOTIFY's */ parse_ok_contact(p, req); |