diff options
-rw-r--r-- | apps/app_voicemail.c | 21 | ||||
-rw-r--r-- | configs/voicemail.conf.sample | 3 |
2 files changed, 17 insertions, 7 deletions
diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c index c78bcaf7c..0abed1eed 100644 --- a/apps/app_voicemail.c +++ b/apps/app_voicemail.c @@ -8301,14 +8301,21 @@ static struct ast_vm_user *find_or_create(const char *context, const char *mbox) struct ast_vm_user *vmu; AST_LIST_TRAVERSE(&users, vmu, list) { - if (ast_test_flag((&globalflags), VM_SEARCH) && !strcasecmp(mbox, vmu->mailbox)) - break; - if (context && (!strcasecmp(context, vmu->context)) && (!strcasecmp(mbox, vmu->mailbox))) - break; + if (ast_test_flag((&globalflags), VM_SEARCH) && !strcasecmp(mbox, vmu->mailbox)) { + if (strcasecmp(vmu->context, context)) { + ast_log(LOG_WARNING, "\nIt has been detected that you have defined mailbox '%s' in separate\ + \n\tcontexts and that you have the 'searchcontexts' option on. This type of\ + \n\tconfiguration creates an ambiguity that you likely do not want. Please\ + \n\tamend your voicemail.conf file to avoid this situation.\n", mbox); + } + ast_log(LOG_WARNING, "Ignoring duplicated mailbox %s\n", mbox); + return NULL; + } + if (!strcasecmp(context, vmu->context) && !strcasecmp(mbox, vmu->mailbox)) { + ast_log(LOG_WARNING, "Ignoring duplicated mailbox %s in context %s\n", mbox, context); + return NULL; + } } - - if (vmu) - return vmu; if (!(vmu = ast_calloc(1, sizeof(*vmu)))) return NULL; diff --git a/configs/voicemail.conf.sample b/configs/voicemail.conf.sample index a712555cc..b29ae6ca0 100644 --- a/configs/voicemail.conf.sample +++ b/configs/voicemail.conf.sample @@ -221,6 +221,9 @@ sendvoicemail=yes ; Allow the user to compose and send a voicemail while inside ; searchcontexts=yes ; Current default behavior is to search only the default context ; if one is not specified. The older behavior was to search all contexts. ; This option restores the old behavior [DEFAULT=no] + ; Note: If you have this option enabled, then you will be required to have + ; unique mailbox names across all contexts. Otherwise, an ambiguity is created + ; since it is impossible to know which mailbox to retrieve when one is requested. ; callback=fromvm ; Context to call back from ; if not listed, calling the sender back will not be permitted ; exitcontext=fromvm ; Context to go to on user exit such as * or 0 |