[kamailio/kamailio] "debug: previous udp send returned non zero" for every log message being sent to remote server with log_custom (#2452)
21 Aug
2020
21 Aug
'20
3:38 a.m.
https://github.com/kamailio/kamailio/blob/ae23876b3091db2e62f651e05fa695d29…
This function appears to be determining, for an unknown reason, that the log message that
was sent immediately prior (successfully, I might add) to my logstash server via UDP
returned non-zero from the udp_send
I am receiving the message and the subsequent error message just fine, so I am unsure of
what the issue could be.
kamailio 5.4.0 (x86_64/linux) 19128f
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST,
DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY,
USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR,
USE_DST_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535,
DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: 19128f
compiled on 20:14:01 Aug 19 2020 with gcc 4.8.5
CentOS Linux release 7.8.2003 (Core)
x86_64
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2452
21 Aug
21 Aug
10:10 p.m.
After further research, it appears that udp_send() returns the result of sendto() which to
my understanding returns the number of bytes sent, so it will always be non-zero on a
successful send.
It looks like back on Jan 19th the logging message was changed from LG_DBG to udp_send so
I'm not sure if that was intentional or maybe an oversight?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2452#issuecomment-678447059
24 Aug
24 Aug
11:08 a.m.
Thanks for digging further into it. The change was done to avoid looping back to the same
function if the `log_custom` was set as custom logging backend in the core. However, if
udp_send() fails for first message, likely to fail for saying the previous execution step
failed. So I changed that to printing to stderr directly, just to have a way to try to
troubleshoot such cases.
And indeed, send_udp() returns the value of the sendto() and should be positive in case of
a successful operation.
The fixes are in master and 5.4 branches -- reference commit:
f16d046f6330f897b374d9bb7dab3b2ce8eab3a7 .
Test and if still issues, reopen.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2452#issuecomment-678976694
11:08 a.m.
Closed #2452.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2452#event-3684247636
1556
days inactive
1559
days old
3 comments
2 participants
participants (2)
-
brooksbridges-call48 -
Daniel-Constantin Mierla