Commit cef63b761f for asterisk.org
commit cef63b761f437eb4c8e3d7e26b74485f3d91c7eb
Author: phoneben <3232963@gmail.com>
Date: Sun Apr 26 23:34:00 2026 +0300
app_queue: Fix raise_respect_min lost in copy_rules() breaking rN queue rules
app_queue: Fix raise_respect_min not copied in copy_rules() causing rN rules to be ignored.
`copy_rules()` never copied `raise_respect_min` into the per-call rule list, so the flag was always 0 when a timed penaltychange rule fired, making `rN` behave like plain `N` and raising members below `min_penalty` that should have been excluded.
Also fixes `update_qe_rule()` not propagating the flag from `qe->pr` to `qe`, and dropping the `r` prefix when saving back to `QUEUE_RAISE_PENALTY`.
Resolves: #1901
diff --git a/apps/app_queue.c b/apps/app_queue.c
index b8ea7a616e..06fc4b0ac1 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -6049,7 +6049,12 @@ static void update_qe_rule(struct queue_ent *qe)
raise_penalty = max_penalty;
}
- snprintf(raise_penalty_str, sizeof(raise_penalty_str), "%d", raise_penalty);
+ qe->raise_respect_min = qe->pr->raise_respect_min;
+ if (qe->raise_respect_min) {
+ snprintf(raise_penalty_str, sizeof(raise_penalty_str), "r%d", raise_penalty);
+ } else {
+ snprintf(raise_penalty_str, sizeof(raise_penalty_str), "%d", raise_penalty);
+ }
pbx_builtin_setvar_helper(qe->chan, "QUEUE_RAISE_PENALTY", raise_penalty_str);
qe->raise_penalty = raise_penalty;
ast_debug(3, "Setting raised penalty to %d for caller %s since %d seconds have elapsed\n",
@@ -8737,6 +8742,7 @@ static void copy_rules(struct queue_ent *qe, const char *rulename)
new_pr->max_relative = pr_iter->max_relative;
new_pr->min_relative = pr_iter->min_relative;
new_pr->raise_relative = pr_iter->raise_relative;
+ new_pr->raise_respect_min = pr_iter->raise_respect_min;
AST_LIST_INSERT_TAIL(&qe->qe_rules, new_pr, list);
}
}