---------- Forwarded message --------- From: Enrico Bandiera enrico.bandiera@cloud.timenet.it Date: mer 13 feb 2019 alle ore 09:18 Subject: Re: [SR-Users] pv_buffer_size config value actually ignored? To: Daniel-Constantin Mierla miconda@gmail.com
Hello Daniel, yes we do use that $avp for a log line which is actually a big json, the $avp is composed by multiple jansson_append calls and at the end an xlog() calls prints it Could it also be that I'm miscalculating because of the encoding? 4K utf8 characters could be more than 4K (but I don't think there are so many multibytes characters to let it reach the 8K limit)
Il giorno mer 13 feb 2019 alle ore 09:15 Daniel-Constantin Mierla < miconda@gmail.com> ha scritto:
Hello,
I pushed a patch to print more details about printed size, current token size and overall output buffer size in those error log messages -- the commit:
* https://github.com/kamailio/kamailio/commit/44e48809a608d4de46d61198137cd669...
Can you try with this patch and send again here on mailing list the error log messages?
Cheers, Daniel
On 13.02.19 09:20, Enrico Bandiera wrote:
Hello Daniel, here are the logs:
29(46) ERROR: <core> [core/pvapi.c:1457]: pv_printf(): no more space for spec value - printed:7 token:5131 buffer:4096 29(46) ERROR: <core> [core/pvapi.c:1467]: pv_printf(): buffer overflow -- increase the buffer size... 29(46) ERROR: <core> [core/pvapi.c:1457]: pv_printf(): no more space for spec value - printed:7 token:5129 buffer:4096 29(46) ERROR: <core> [core/pvapi.c:1467]: pv_printf(): buffer overflow -- increase the buffer size...
Thanks, Enrico
Il giorno mer 13 feb 2019 alle ore 09:45 Daniel-Constantin Mierla < miconda@gmail.com> ha scritto:
Hi, could it be caused by this?
https://github.com/kamailio/kamailio/blob/master/src/modules/xlog/xlog.c#L65
I tried to increment it, recompiled kamailio and the error doesn't show up anymore
Thanks, Enrico
Il giorno mer 13 feb 2019 alle ore 17:29 Enrico Bandiera < enrico.bandiera@cloud.timenet.it> ha scritto:
Hello,
you actually need to set the modparam for it:
- https://www.kamailio.org/docs/modules/stable/modules/xlog.html#xlog.p.buf_si...
xlog module is using its own buffer, not the one from pv core file.
Cheers, Daniel
On 13.02.19 21:36, Enrico Bandiera wrote:
OK - probably we should add in the wiki at pv_buffer_size a note to say that for xlog messages, there is a dedicated modparam to set the internal buffer size.
Cheers, Daniel
On 14.02.19 09:00, Enrico Bandiera wrote: