From 6436d82654f96fdde6009b80de4cdc8fff6f1c9e Mon Sep 17 00:00:00 2001 From: Trevor Peirce Date: Tue, 7 Jun 2022 14:53:34 -0700 Subject: [PATCH] res_pjsip: Actually enable session timers when timers=always When a pjsip endpoint is defined with timers=always, this has been a functional noop. This patch correctly sets the feature bitmap to both enable support for session timers and to enable them even when the endpoint itself does not request or support timers. ASTERISK-29603 Reported-By: Ray Crumrine Change-Id: I8b5eeaa9ec7f50cc6d96dd34c2b4aa9c53fb5440 --- res/res_pjsip/pjsip_configuration.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/res/res_pjsip/pjsip_configuration.c b/res/res_pjsip/pjsip_configuration.c index 39c3daba5a..ace68d2fdc 100644 --- a/res/res_pjsip/pjsip_configuration.c +++ b/res/res_pjsip/pjsip_configuration.c @@ -220,7 +220,7 @@ static int timers_handler(const struct aco_option *opt, struct ast_variable *var } else if (!strcasecmp(var->value, "required")) { endpoint->extensions.flags |= PJSIP_INV_REQUIRE_TIMER; } else if (!strcasecmp(var->value, "always") || !strcasecmp(var->value, "forced")) { - endpoint->extensions.flags |= PJSIP_INV_ALWAYS_USE_TIMER; + endpoint->extensions.flags |= (PJSIP_INV_SUPPORT_TIMER | PJSIP_INV_ALWAYS_USE_TIMER); } else if (!ast_false(var->value)) { return -1; }