[Asterisk-code-review] stun: Emit warning message when STUN request times out (asterisk[master])

[Sébastien Duthil]

Sébastien Duthil has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16144 )


Change subject: stun: Emit warning message when STUN request times out
......................................................................

stun: Emit warning message when STUN request times out

Without this message, it is not obvious that the reason is STUN timeout.

ASTERISK-29507 #close

Change-Id: I26e4853c23a1aed324552e1b9683ea3c05cb1f74
---
M main/stun.c
1 file changed, 2 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/44/16144/1

diff --git a/main/stun.c b/main/stun.c
index e917944..4c3dc93 100644
--- a/main/stun.c
+++ b/main/stun.c
@@ -438,6 +438,7 @@
 			ms = ast_remaining_ms(start, 3000);
 			if (ms <= 0) {
 				/* No response, timeout */
+				ast_log(LOG_WARNING, "STUN request number %d timed out.\n", retry);
 				res = 1;
 				continue;
 			}
@@ -448,6 +449,7 @@
 			}
 			if (!res) {
 				/* No response, timeout */
+				ast_log(LOG_WARNING, "STUN request number %d timed out.\n", retry);
 				res = 1;
 				continue;
 			}

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16144
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I26e4853c23a1aed324552e1b9683ea3c05cb1f74
Gerrit-Change-Number: 16144
Gerrit-PatchSet: 1
Gerrit-Owner: Sébastien Duthil <sduthil at wazo.community>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210705/f13c647e/attachment.html>