[Asterisk-code-review] res_pjsip_sdp_rtp: Do not warn on unknown sRTP crypto suites. (asterisk[master])
Friendly Automation
asteriskteam at digium.com
Mon Dec 6 10:57:41 CST 2021
Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/17591 )
Change subject: res_pjsip_sdp_rtp: Do not warn on unknown sRTP crypto suites.
......................................................................
res_pjsip_sdp_rtp: Do not warn on unknown sRTP crypto suites.
res_sdp_crypto_parse_offer(.) emits many log messages already.
ASTERISK-29785
Change-Id: I1a191ebe4fec1102946d4e31887e5197ca02dfe8
---
M res/res_pjsip_sdp_rtp.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Kevin Harwell: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved
Friendly Automation: Approved for Submit
diff --git a/res/res_pjsip_sdp_rtp.c b/res/res_pjsip_sdp_rtp.c
index a3bc462..ae200a7 100644
--- a/res/res_pjsip_sdp_rtp.c
+++ b/res/res_pjsip_sdp_rtp.c
@@ -1144,7 +1144,7 @@
return 0;
}
- ast_log(LOG_WARNING, "Ignoring crypto offer with unsupported parameters: %s\n", crypto_str);
+ ast_debug(1, "Ignoring crypto offer with unsupported parameters: %s\n", crypto_str);
}
/* no usable crypto attributes found */
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17591
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I1a191ebe4fec1102946d4e31887e5197ca02dfe8
Gerrit-Change-Number: 17591
Gerrit-PatchSet: 2
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211206/9eaa79f4/attachment.html>
More information about the asterisk-code-review
mailing list