Daniel-Constantin Mierla writes:
> @juha-h - can you try with latest master again? I found an issue in
> ctl module that was introduced in master branch a while ago, but after
> 5.4. It may be that one. If it is not fixed, then get the `kamctl
> trap` after it hangs, to see where the processes are
Module: kamailio
Branch: master
Commit: 2664a51c35a1e8b5adc4c7e6cedde60bc01f0b28
URL:
https://github.com/kamailio/kamailio/commit/2664a51c35a1e8b5adc4c7e6cedde60bc01f0b28
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2021-02-11T17:27:44+02:00
Fixed typo.
---
Modified: src/modules/rr
Daniel-Constantin Mierla writes:
> rr: use rr params with record_route_preset()
>
> - param buffer was checked, but not used
Thanks. Tested and now add_rr_param() has effect on R-R headers added by
record_route_preset(). add_rr_param() can be called both before and
after record_route_preset().
Module: kamailio
Branch: master
Commit: 857e9deb4c2455ee2ea0c05446da54f97e1f0a63
URL:
https://github.com/kamailio/kamailio/commit/857e9deb4c2455ee2ea0c05446da54f97e1f0a63
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2021-02-11T17:44:19+02:00
Improved README.
---
Modified: src
Daniel-Constantin Mierla writes:
> core: execute event_route[core:worker-one-init] for sctp or tcp
>
> - if no udp worker is started, the event route is executed by next
> available transport, in the order: sctp, tcp (or tls)
Thanks, Juha
___
Kamailio
Daniel-Constantin Mierla writes:
> I was just about to write you on the other thread to test and see if
> works, as I did the development but without having the time to test.
I tested with tcp and tls listeners
Mar 12 20:16:43 sip-proxy[21780]: Listening on
Mar 12 20:16:43 sip-proxy[21780]:
I got latest master properly installed and gave ca_path a try.
I placed four ca certs in directory /etc/sip-proxy/certs/ca_list:
# ls /etc/sip-proxy/certs/ca_list
class3_X0E.crt dst_root_ca_x3.pem lets-encrypt-x3-cross-signed.pem
root_X0F.crt
Then I created file ca_list.pem that contained al
Daniel-Constantin Mierla writes:
> I pushed a commit to make that error message print the file name. Try
> again with latest master and paste here again the new error message to
> see what's there.
Mar 23 13:41:37 lohi /usr/bin/sip-proxy[7261]: ERROR: tls [tls_domain.c:604]:
load_ca_list(): TLSs
Daniel-Constantin Mierla writes:
> Pushed a commit to fix the error message.
Now kamailio starts, but when baresip sip client tries to connect over
TLS, I get these errors:
Mar 23 14:02:23 lohi /usr/bin/sip-proxy[1732]: ERROR: tls [tls_server.c:1283]:
tls_h_read_f(): protocol level error
Mar 23
Daniel-Constantin Mierla writes:
> Author: Daniel-Constantin Mierla
> Committer: Daniel-Constantin Mierla
> Date: 2021-04-26T11:22:20+02:00
>
> core/mem: q_malloc - use #else with DBG_QM_MALLOC for debug on freed fragments
The large number of log messages are still there, but perhaps this was
Daniel,
I studied a bit topos_redis source and noticed that it lacks x_context
td and tt keys.
Is there something else missing in order to make spirals work also when
redis backend is in use?
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-de
Daniel-Constantin Mierla writes:
> The idea was to try to optimize and tailor it for redis specific
> operation, otherwise you can try with topos+db_redis, which should give
> a more SQL-like approach and the columns in mysql tables should appear
> as attributes inside redis db. The db_redis was a
Daniel-Constantin Mierla writes:
> Regarding topos_redis, I just pushed a commit to it in master branch,
> can you give it another try?
Still when A receives 183 Session Progress from P1 and send PRACK, P1
rejects it with 404 Not Found (pcap enclosed). Didn't have debug on,
but can do if needed.
This used to work:
$var(count) = $(var(servers){param.count,%});
but latest master gives:
Jun 1 16:02:04 lab sip-proxy[11965]: 0(12020) INFO: pv [pv_shv.c:60]:
shvar_init_locks(): locks array size 16
Jun 1 16:02:04 lab sip-proxy[11965]: 0(12020) ERROR: pv [pv_trans.c:3254]:
tr_parse_para
Daniel-Constantin Mierla writes:
> Give it a try again with latest master, I did also some basic tests and
> seems ok.
Thanks, works now.
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bi
C (gcc) [M outbound.so] outbound_mod.o
outbound_mod.c: In function 'mod_init':
outbound_mod.c:112:25: warning: pointer targets in passing argument 1 of 'SHA1'
differ in signedness [-Wpointer-sign]
SHA1(flow_token_secret.s, flow_token_secret.len, ob_key.s);
~^~
In
Michael Furmur writes:
> https://github.com/kamailio/kamailio/pull/2812
Thanks, Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
I tested xflags by making these calls in a route block:
INFO(** setting flag 19);
setflag(19);
INFO(** setting xflag 19);
setxflag(19);
and these calls in onreply_route block:
if (isflagset(19))
INFO(* flag 19 is set);
else
INFO(***
Daniel-Constantin Mierla writes:
> I pushed a patch for it -- onreply_route was using the xflags from reply
> structure, not from transaction.
Thanks, it works now,
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https:
Juha Heinanen writes:
> While testing xflags, i noticed that a regular flag that I set AFTER
> calling t_newtrans() stays set in onreply_route even when I do not
> call t_flush_flags().
I made the same test with xflags and they do require t_flush_xflags()
call if an xflag is set after t
Daniel-Constantin Mierla writes:
> Afaik, flags after t_newtran() were supposed not to be moved to
> transaction if t_flush_flags(), that being the purpose of the later
> function. Are you doing any other tm operations between creating the new
> transaction and relaying the request or end of scrip
Daniel-Constantin Mierla writes:
> The purpose of that t_flush_flags() is documented:
>
> *
> https://www.kamailio.org/docs/modules/stable/modules/tmx.html#tmx.f.t_flush_flags
>
> The role was to push the flags set after t_newtran() to
> transaction. If that is not happening/is not needed for
README has:
In the same way, rule_id_avp, if defined,
contains the id of the rule that selected each gateway.
It should read "selected the gateway".
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamail
CC (gcc) [M db_mysql.so]my_cmd.o
my_cmd.c: In function 'set_field':
my_cmd.c:774:16: warning: assignment from incompatible pointer type
[-Wincompatible-pointer-types]
bind->is_null = &f->is_null;
^
my_cmd.c: In function 'bind_result':
my_cmd.c:1004:21: warning: as
Henning Westerholt writes:
> at least in my installation with mariadb it's now fixed in current
> master. Let me know if you still see it.
The warnings are now gone, thanks,
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.
When building debian package from master, I noticed this kind of error
message:
s ../../../doc/stylesheets/serdoc2man.xsl auth.xml
make[3]: s: Command not found
../../Makefile.modules:283: recipe for target 'auth.7' failed
The line in Makefile.modules is here:
$(MOD_NAME).7: $(MOD_NAME).xml
Daniel-Constantin Mierla writes:
> Grepping for that variable in the makefiles reveals that it looks for
> docbook2x-man or db2x_docbook2man.
>
> Likely not available in your system, so either they were part of a
> installed package or related dependency in the past, or just a new
> package that
Juha Heinanen writes:
> I added that package and the error was gone, but a new ones were
> produced:
>
> /usr/bin/docbook2x-man -s ../../../doc/stylesheets/serdoc2man.xsl auth.xml
> auth.xml:7: warning: failed to load external entity
> "../../../../doc/docbook/entitie
Module: kamailio
Branch: master
Commit: 26449dc2da3b47dd71265e8103de5957bd2245f4
URL:
https://github.com/kamailio/kamailio/commit/26449dc2da3b47dd71265e8103de5957bd2245f4
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2018-08-23T11:53:34+03:00
modules/auth: fixed path in auth.xml
CC (gcc) [M uac.so] uac_reg.o
uac_reg.c: In function 'rpc_uac_reg_active':
uac_reg.c:2110:6: warning: unused variable 'none' [-Wunused-variable]
str none = {"none", 4};
^~~~
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.k
When I execute pua.publish over jsonrpc, I get error at reply:
Sep 10 09:37:34 char /usr/bin/sip-proxy[11104]: INFO: Executing JSON request
<{"jsonrpc":"2.0","method":"pua.publish","params":[...],"id":1}>
Sep 10 09:37:34 char /usr/bin/sip-proxy[11079]: ERROR: jsonrpcs
[jsonrpcs_mod.c:778]: json
Daniel-Constantin Mierla writes:
> Can you try with latest master branch and attach all logs printed with
> debug=3 in kamailio.cfg?
I got this:
Sep 11 21:47:33 char /usr/bin/sip-proxy[3149]: DEBUG:
[core/tcp_read.c:1759]: handle_io(): received n=8 con=0x7f2f544fe450, fd=12
Sep 11 21:47:33
Module: kamailio
Branch: 5.1
Commit: 53bc3a936d43d0eb23c61b0d1f089c63b437b0e9
URL:
https://github.com/kamailio/kamailio/commit/53bc3a936d43d0eb23c61b0d1f089c63b437b0e9
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2018-09-17T18:17:36+03:00
core: backported commit
I have one local module and updated its module interface to the new
one. Then I build K, started it and got a crash. I thought that it
must my mistake in the module interface update, but when I looked at the
core dump, it did point to another module:
(gdb) where
#0 0x5643e66fd349 in registe
Daniel-Constantin Mierla writes:
> I just pushed a patch for it, can you test and see if all goes fine with
> your config?
cfg_rpc module error is now gone, but got another one:
Oct 1 08:37:19 char /usr/bin/sip-proxy[26419]: ERROR: [db.c:419]:
db_table_version(): invalid type (8) or nul (0) v
Daniel-Constantin Mierla writes:
> This was related to adding support for UNSIGNED types. So far we had
> only signed types in DB API, but latest MySQL version is inserting 0
> without any error when the actual value is a signed negative value.
>
> Should be fixed now in latest master, try with i
I don't have any negative values in any of the lcr tables.
-- Juha
> Will review all database table definitions and update accordingly. Once
> is finished, I will post an update.
>
> Cheers,
> Daniel
>
>
> On 01.10.18 08:59, Juha Heinanen wrote:
> > Daniel-Cons
Daniel-Constantin Mierla writes:
> But as soon as a value goes negative, things get broken. For example,
> with the modules storing expire/lifetime (e.g., usrloc, dialog) being
> timestamp + interval, if interval is very long, then the value goes over
> mid of UNSIGNED, practically becoming negati
Daniel-Constantin Mierla writes:
> Given that it may take some time to propagate the new unsigned types, I
> added a paramter to db_mysql to enable/disable converting to these
> types. By default is off, so it should work as before last Friday.
>
> Can you give another try with latest master and
Daniel-Constantin Mierla writes:
> This update with new database types DB1_UINT and DB1_UBIGINT started on
> Friday. I expected that only insert/updates with high values to be
> affected, but it seems that loading is also hit.
So, for example, below DB1_INT should be changed to DB1_UNIT and VAL_I
Daniel-Constantin Mierla writes:
> You have to change:
>
> if(VAL_TYPE(ROW_VALUES(row) + 11) != DB1_INT) {
>
> to something like:
>
> if(VAL_TYPE(ROW_VALUES(row) + 11) != DB1_INT && VAL_TYPE(ROW_VALUES(row) +
> 11) != DB1_UINT) {
Why do I need to test both DB1_INT and DB1_UINT, since the colu
Module: kamailio
Branch: master
Commit: dd9615e7b0e20a2f63f380ee16837def985676aa
URL:
https://github.com/kamailio/kamailio/commit/dd9615e7b0e20a2f63f380ee16837def985676aa
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2018-10-01T20:43:43+03:00
modules/lcr: tried to test unsigned int
K nowadays has a large number of modules. Some of them may not anymore
be in use by anybody and could be deleted. One example is peering
module that I wrote in 2008 for a demo. I suspect that it has not been
used since then.
Perhaps we could issue a list of suspects and if then, if no one
objec
Got this today with latest master:
CC (gcc) [M sdpops.so] sdpops_mod.o
sdpops_mod.c: In function 'sdp_get_sess_version':
sdpops_mod.c:1871:14: warning: unused variable 'sdp' [-Wunused-variable]
sdp_info_t *sdp = NULL;
^~~
It would be nice if people who make commits would
Daniel-Constantin Mierla writes:
> I haven't used peering at all, so I do not have any comment about its
> usage, but for at least one major version should be marked obsolete. You
> can add it as a note in the docs (at the top of the readme for the
> module, in the overview section) and if no one
Module: kamailio
Branch: master
Commit: 7e1112abd6949600cfabb163466e1557e5baa296
URL:
https://github.com/kamailio/kamailio/commit/7e1112abd6949600cfabb163466e1557e5baa296
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2018-10-02T13:22:31+03:00
modules/peering: added a note that the
Daniel-Constantin Mierla writes:
> It is a matter of the OS, compiler and version if this kind of warning
> is shown with the default flags. For example clang doesn't show it.
> Latest versions of gcc do it.
If clang does not show it, then the compile test should be done with
gcc. It is very bad
Mikko Lehto writes:
> Thanks for the feedback.
> Looks like -Wall was not enabled on clang, I should have noticed that long
> time ago.
>
> I just enabled -Wall locally and try to see if anything interesting
> comes up that I could help fix.
Mikko,
Thank you very much. Clean compilation avoid
Daniel-Constantin Mierla writes:
> sl: new pv $ltt(key) - return local generated to tag
There was this kind of notice on Oct 1:
a short reminder that the code for next major release v5.2.0 will be
frozen at the end of Thursday this week, October 4.
If anyone has new features that they wan
A small typo here:
2.1. Kamailio Modules
The module depends on the following modules (in the other words the
listed modules must be loaded before this module):
* tm - accounting module
A question:
Is rms_play() done asynchronously or does it reserve the K process that
handled the req
Julien Chavanton writes:
> No Kamailio processes are blocking, one thread is started for each call /
> media stream the thread is running a "ticker" that is processing all the
> samples in the audio streams.
So just to clarify, if Kamailio is started, for example, with -n 8, it
does not limit the
One more question: at this stage, rtp_media_server is not able to play
early audio?
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Julien Chavanton writes:
> However, bridging early media may be suffisant, in the end "faking" a
> remote ringing may not be a very good user experience. But maybe we want to
> play a customized ring back tone.
One application that I had in mind is to play some kind of announcement
in early media
I would suggest the following diff that allows the script to learn that
timestamp in ephemeral auth username has expired. The script can then
tell it to user via a suitable response.
-- Juha
diff --git a/src/modules/auth/api.h b/s
I'm getting a few compiler warnings when building latest master on
Debian 11.
-- Juha
---
CC (gcc) [M dispatcher.so] ds_ht.o
In file included from ../../core/mem/shm.h:43,
from ../../core/mem/shm_mem.h:32,
from ds_ht.c:25
Latest K master crashes on Debian 11 when handling SUBSCRIBE message.
(Earlier messages didn't go through when there was word subscribe in
subject.)
-- Juha
--
(gdb) bt
#0 0x7f58c362cad0 in trim_leading (_s=0x7f58bceeb558) at
../../core/trim.h:59
#1 0x7f58c362c
Daniel-Constantin Mierla writes:
> Hopefully the dispatcher and jsonrpcs warnings are fixed. I guess that
> local.so is a custom module, you have to change the logging/printf()
> specifiers from %d to%ld because some internal fields in kamailio were
> changed from int to long int for a better supp
Daniel-Constantin Mierla writes:
> I will check it, but in general the kemi function is executed by the
> native kamailio.cfg function, after it evaluates its parameters with
> kamailio pseudo-variables. It is nothing specific for the kemi
> function than just needing the string/int values of the
Daniel-Constantin Mierla writes:
> I am for it, maybe the next major release will be bumped to 6.0, so
> going with clang formatting would be good because makes the readability
> better.
Which style? https://clang.llvm.org/docs/ClangFormatStyleOptions.html
supports many:
LLVM A style complying
I guess I missed the clang-format discussion or did it happen?
Fr example, I would have voted for
SpaceBeforeParens: ControlStatements
-- Juha
___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
Kamailio 5.7 build on Debian Bookworm gave me the following build
warnings. The only curl related build dependency that I have is
libcurl4-openssl-dev and the only openssl related build dependency is
libssl-dev.
-- Juha
--
CC (gcc) [M http_client.so]
Victor Seva writes:
> @juha-h can you please test https://github.com/kamailio/kamailio/pull/3482?
> At least the OPENSSL_fork_* warnings should be solved.
I gave the PR a try and now got the warnings below (no fork warnings
anymore).
-- Juha
tls_mod.c: In funct
I'm not sure if I can participate, but one issue that could be discussed
is how to distribute load to many worker processes in situations when one
Kamailio forwards requests to another over TCP or TLS.
If I have understood correctly, by default, sending Kamailio reuses
existing TCP or TLS connecti
Andreas Granig writes:
> The new keepalive module states that it "performs destinations
> monitoring either for itself, or on the behalf of other modules".
Regarding "on behalf of other modules", I didn't find any API function
documentation in README file.
-- Juha
__
Why is all K related github messages also sent to sr-dev list? There is
lots of duplicate traffic because of it if sr-dev member is also
watching github notifications.
Or is it so that sr-dev members are not supposed to subscribe to
kamailio github notifications?
-- Juha
___
Daniel-Constantin Mierla writes:
> Several years ago there was a discussion on whether to use another
> mailing list and the decision was to stick to one as people can set
> their own filters to sort or drop messages. I am still fine with this, I
> can easily search and correlate related discussio
LCR module load_gws() call can currently filter matching gateways based
on regular expression match of From URI. I was thinking to add as an
alternative to the regular expression match a match based on mt_match()
of mtree module.
That would require that mtree module provides an API function
corre
Module: kamailio
Branch: master
Commit: 556fab89cec8d1eda726f38f4ddf0477e97bb79d
URL:
https://github.com/kamailio/kamailio/commit/556fab89cec8d1eda726f38f4ddf0477e97bb79d
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-05-15T16:40:28+03:00
modules/mtree: added mt_match API function
Module: kamailio
Branch: master
Commit: c5127e6d19c4f30072d81213f5eb49e4a6306bb3
URL:
https://github.com/kamailio/kamailio/commit/c5127e6d19c4f30072d81213f5eb49e4a6306bb3
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-05-15T18:12:32+03:00
modules/lcr: added possibility to match
wiki Log In link leads to unprotected (http) page
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Module: kamailio
Branch: master
Commit: c4754f60f010b8ad55a3785ea362dfeeda008300
URL:
https://github.com/kamailio/kamailio/commit/c4754f60f010b8ad55a3785ea362dfeeda008300
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-05-15T19:02:21+03:00
modules/lcr: forgot to copy new versions of
Module: kamailio
Branch: master
Commit: 7386b03b5e70b9faf61d84bfabf88092afc65a8c
URL:
https://github.com/kamailio/kamailio/commit/7386b03b5e70b9faf61d84bfabf88092afc65a8c
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-05-15T19:04:48+03:00
Merge branch 'master' of https://
Module: kamailio
Branch: master
Commit: bb517cda1ae9ea0229195871338b0967a9b0a7fc
URL:
https://github.com/kamailio/kamailio/commit/bb517cda1ae9ea0229195871338b0967a9b0a7fc
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-05-16T08:26:22+03:00
modules/lcr: improved documentation
Today when building K 5.0 on Debian Jessie, I got this compiler warning
that I don't remember seeing before:
CC (gcc) [M domain.so] domain.o
domain.c: In function 'w_lookup_domain':
domain.c:222:5: warning: 'prefix' may be used uninitialized in this function
[-Wmaybe-uninitialized]
if
Daniel-Constantin Mierla writes:
> Fixed now.
OK, thanks, Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
How about backporting this fix to 5.0?
https://github.com/kamailio/kamailio/commit/8e185ddf8e81af8d6283a87ec95116965235752a
-- Juha
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listi
Module: kamailio
Branch: 5.0
Commit: d49131e05033a78585bebea1bce503e97274a24e
URL:
https://github.com/kamailio/kamailio/commit/d49131e05033a78585bebea1bce503e97274a24e
Author: Dmitri Savolainen
Committer: Juha Heinanen
Date: 2017-06-26T18:06:39+03:00
presence: subsription double delete fix
I got these TLS related warnings when compiling K 5.0 on Debian Stretch.
-- Juha
CC (gcc) [M tls.so] tls_init.o
tls_init.c: In function 'init_ssl_methods':
tls_init.c:377:2: warning: 'TLSv1_client_method' is deprecated
[-Wdeprecated-declarations]
ssl_methods[TLS_USE_TLSv1_cli - 1]
Donald Carr writes:
> @miconda, I have written a routine that puts the parameters for the
> tel: uri into an array with key/value separated, then uses qsort to
> sort. With this, they can be put into the sip: uri in correct sorted
> order. The sort is very esoteric, specific to the requirements f
Donald Carr writes:
> At the link below, you can go to 19.1.6. It is a very strange ordering
> giving priority to two parameters that should be first, if present.
>
> https://www.ietf.org/rfc/rfc3261.txt
>
> 19.1.6 Relating SIP URIs and tel URLs
> . . . .
> ```
>To mitigate this problem, el
Daniel-Constantin Mierla writes:
> The warnings were reported before:
>
> - https://lists.kamailio.org/pipermail/sr-dev/2017-April/040137.html
>
> Meanwhile (recently) I fixed them in master branch (see git log for
> src/modules/tls/) and I am fine to backport provided that someone does
> some
Donald Carr writes:
> Are you interested in the fix? I agree it is a bit esoteric, but, I
> think worth it to help reduce problems in the future. Especially
> removing the double domain name in the sip uri, and removing visual
> separators from the phone-context when it is a global number. Well,
>
Module: kamailio
Branch: 5.0
Commit: 25defde62284bd23fc41ec4276bb11c0c6ece687
URL:
https://github.com/kamailio/kamailio/commit/25defde62284bd23fc41ec4276bb11c0c6ece687
Author: Daniel-Constantin Mierla
Committer: Juha Heinanen
Date: 2017-07-19T13:25:04+03:00
tm: copy branch flags in the new
CC (gcc) [M rls.so] subscribe.o
subscribe.c: In function 'reply_200':
subscribe.c:340:6: warning: unused variable 'len' [-Wunused-variable]
int len;
^~~
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://l
Module: kamailio
Branch: master
Commit: b325a7711ac405e5598f7fbd224fb1b8e84221cf
URL:
https://github.com/kamailio/kamailio/commit/b325a7711ac405e5598f7fbd224fb1b8e84221cf
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-08-25T10:29:44+03:00
modules/lcr: fixed testing if
Module: kamailio
Branch: 5.0
Commit: ba6ec1a9c651e43d4a0e89724ad5a7dfa6dafda7
URL:
https://github.com/kamailio/kamailio/commit/ba6ec1a9c651e43d4a0e89724ad5a7dfa6dafda7
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-08-25T10:32:26+03:00
modules/lcr: fixed testing if
Module: kamailio
Branch: master
Commit: b0f941e1ef9a013472ea42bade3c8ecae9111768
URL:
https://github.com/kamailio/kamailio/commit/b0f941e1ef9a013472ea42bade3c8ecae9111768
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-09-06T21:34:56+03:00
modules/lcr: improved description of
Module: kamailio
Branch: master
Commit: 06a5bb4cff0319a19660b711e7d987b30a939345
URL:
https://github.com/kamailio/kamailio/commit/06a5bb4cff0319a19660b711e7d987b30a939345
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-09-11T22:21:26+03:00
modules/lcr: remove extra prefix_column
I tried latest Linphone and when it tried to register, K complained:
Sep 13 20:56:06 box /usr/bin/sip-proxy[111574]: WARNING: registrar
[sip_msg.c:194]: check_contacts(): contact uri is too long:
[sip:username@host:port;app-id=929724111839;pn-type=firebase;pn-tok=dYqC3AnzCxI:APA91bF84yDlViEZtJyu
It turned out that registrar module takes max contact size from
contact_max_size param that defaults to 255.
Would it make sense to increase the default (and location table contact
column size) for the next K release so that K would work out-of-the-box
with e.g. Linphone?
-- Juha
___
Looks like also the SIP world is getting more complex all the time and
URIs are getting larger.
How about if I increase size of location table contact column from 255
to 386 and make the same change in registrar module contact_max_size
param default value?
I checked and didn't find any other code
Daniel-Constantin Mierla writes:
> if such change to increase the default is to be done, I think it is
> better to double it, rather that just adding 128 which can be hit soon
> again.
OK.
> I haven't checked, but probably the db table column type needs to be
> changed, iirc, varchar was up to 2
Daniel-Constantin Mierla writes:
> The values are defined inside entities.xml in the same folder. I would
> put there contact_len as a new entity defined for 512, because I think
> we need to adjust also for other modules that store contact URI, such as
> presence or dialog, because they will be h
Module: kamailio
Branch: master
Commit: 62ea33a6e9585e25362bd669c2f1cd11321dc119
URL:
https://github.com/kamailio/kamailio/commit/62ea33a6e9585e25362bd669c2f1cd11321dc119
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-09-14T14:44:41+03:00
increased location table contact field size
Module: kamailio
Branch: master
Commit: 5a1555ca8d52b112b4d4671583aeb0ed44422016
URL:
https://github.com/kamailio/kamailio/commit/5a1555ca8d52b112b4d4671583aeb0ed44422016
Author: Juha Heinanen
Committer: Juha Heinanen
Date: 2017-09-14T14:46:23+03:00
Merge branch 'maste
got these docbook errors today when building latest master on debian
stretch.
-- juha
usr/bin/docbook2x-man -s ../../../doc/stylesheets/serdoc2man.xsl auth.xml
auth.xml:7: warning: failed to load external entity
"../../../../doc/docbook/entities.xml"
%docentities;
^
Entity: line 1:
Daniel-Constantin Mierla writes:
> sipdump: implemented sipdump.enable rpc command
>
> ---
>
> Modified: src/modules/sipdump/sipdump_mod.c
> Modified: src/modules/sipdump/sipdump_write.c
> Modified: src/modules/sipdump/sipdump_write.h
What is sipdump module? Kamailio wiki does not know about i
Daniel-Constantin Mierla writes:
> The module was just pushed earlier today, still work to do on it. The
> online html docs are generated by cron job few times a day, the html
> index is manually updated and not done yet, but the README is already in
> the source code.
OK, Juha
_
CC (gcc) [M sipdump.so] sipdump_mod.o
In file included from ../../core/parser/../ut.h:42:0,
from ../../core/parser/../ip_addr.h:39,
from ../../core/parser/msg_parser.h:37,
from ../../core/sr_module.h:34,
from sipdump_mod.c:
Perhaps 0 would be better choice as "enable" param default value,
since dumping is likely to be done only if some need arises.
What is meaning of wait param, i.e., what is it that is waited for?
3.2. wait (int)
Wait time (microseconds) when no SIP traffic is received.
-- Juha
101 - 200 of 243 matches
Mail list logo