[Asterisk-code-review] res pjsip outbound authenticator digest: Add missing outboun... (asterisk[13])

Matt Jordan asteriskteam at digium.com
Fri May 1 06:55:49 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: res_pjsip_outbound_authenticator_digest: Add missing outbound authenticator callback.
......................................................................


res_pjsip_outbound_authenticator_digest: Add missing outbound authenticator callback.

The Asterisk 13 version of the fix for outbound registration was missing
a key component that set the outbound authenticator's callback that
creates an authenticated request based on an old request. This was
picked up by some outbound registration tests failing in the testsuite.

Change-Id: I5ca9379698c606da36bc38eaffccedaf64211ce3
---
M res/res_pjsip_outbound_authenticator_digest.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/res/res_pjsip_outbound_authenticator_digest.c b/res/res_pjsip_outbound_authenticator_digest.c
index aa35fba..8616b6c 100644
--- a/res/res_pjsip_outbound_authenticator_digest.c
+++ b/res/res_pjsip_outbound_authenticator_digest.c
@@ -158,6 +158,7 @@
 
 static struct ast_sip_outbound_authenticator digest_authenticator = {
 	.create_request_with_auth = digest_create_request_with_auth,
+	.create_request_with_auth_from_old = digest_create_request_with_auth_from_old,
 };
 
 static int load_module(void)

-- 
To view, visit https://gerrit.asterisk.org/317
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5ca9379698c606da36bc38eaffccedaf64211ce3
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list