diff options
author | bweschke <bweschke@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-02-02 21:26:45 +0000 |
---|---|---|
committer | bweschke <bweschke@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-02-02 21:26:45 +0000 |
commit | d66bfd4fc933f739967ca660623dc92076aca3e1 (patch) | |
tree | 7f52e92e0351e5f305102faa13e29ee8ca44415a | |
parent | b14a6878d5c78549074edc1e430344675731ae51 (diff) |
I'm baaaaaaaaaack. :)
Post a warning to the console that things might possibly be misconfigured when queue member's states are still 'Not in Use' when we're about to bridge them with a caller from queue. Also, put some documentation quoted from oej's queues.txt efforts started in /trunk today.
This commit puts #7433 into feedback state for 1.4, and pending no further negative feedback, it will finally be closed.
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@53129 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | UPGRADE.txt | 17 | ||||
-rw-r--r-- | apps/app_queue.c | 4 |
2 files changed, 21 insertions, 0 deletions
diff --git a/UPGRADE.txt b/UPGRADE.txt index 2f90be429..46bf90d06 100644 --- a/UPGRADE.txt +++ b/UPGRADE.txt @@ -163,6 +163,23 @@ Applications: option, it will default to "no" to keep backward compatability with the old behavior. +* Queues depend on the channel driver reporting the proper state + for each member of the queue. To get proper signalling on + queue members that use the SIP channel driver, you need to + enable a call limit (could be set to a high value so it + is not put into action) and also make sure that both inbound + and outbound calls are accounted for. + + Example: + + [general] + limitonpeer = yes + + [peername] + type=friend + call-limit=10 + + * The app_queue application now has the ability to use MixMonitor to record conversations queue members are having with queue callers. Please see configs/queues.conf.sample for more information on this option. diff --git a/apps/app_queue.c b/apps/app_queue.c index 98223946f..22da470cf 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -2637,6 +2637,10 @@ static int try_calling(struct queue_ent *qe, const char *options, char *announce ast_copy_string(oldexten, qe->chan->exten, sizeof(oldexten)); time(&callstart); + if (member->status == AST_DEVICE_NOT_INUSE) + ast_log(LOG_WARNING, "The device state of this queue member, %s, is still 'Not in Use' when it probably should not be! Please check UPGRADE.txt for correct configuration settings.\n", member->membername); + + bridge = ast_bridge_call(qe->chan,peer, &bridge_config); if (strcasecmp(oldcontext, qe->chan->context) || strcasecmp(oldexten, qe->chan->exten)) { |