[sr-dev] is liblua5.3 supported?

debian stretch has liblua5.3. is that version supported by kamailio or is 5.1 still required? -- juha ___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:91a81625: srdb1/schema: made 'domain_attrs_domain_attrs_idx' index not unique

Module: kamailio Branch: master Commit: 91a81625f89da38e5890f3b3c84ed4141eb3652c URL: https://github.com/kamailio/kamailio/commit/91a81625f89da38e5890f3b3c84ed4141eb3652c Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-04-15T17:39:39+

Re: [sr-dev] is liblua5.3 supported?

Daniel-Constantin Mierla writes: > In the Makefile the module tries to detect lua 5.1 or 5.2, so I guess > both should be fine. I haven't tried with 5.3 -- does it give compile > errors? I haven't tried either. Perhaps it better to stick with lua 5.2, since it is still used with lots of other

Re: [sr-dev] [kamailio/kamailio] Incorrect translation of tel: uri to sip: uri in tel2sip() (#1173)

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,

[sr-dev] git:5.0:25defde6: tm: copy branch flags in the new branch created by the dns failover

Module: kamailio Branch: 5.0 Commit: 25defde62284bd23fc41ec4276bb11c0c6ece687 URL: https://github.com/kamailio/kamailio/commit/25defde62284bd23fc41ec4276bb11c0c6ece687 Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-07-19T13

Re: [sr-dev] tls warnings

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 >

Re: [sr-dev] [kamailio/kamailio] Incorrect translation of tel: uri to sip: uri in tel2sip() (#1173)

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,

[sr-dev] tls warnings

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]

Re: [sr-dev] [kamailio/kamailio] Incorrect translation of tel: uri to sip: uri in tel2sip() (#1173)

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

[sr-dev] git:master:556fab89: modules/mtree: added mt_match API function

Module: kamailio Branch: master Commit: 556fab89cec8d1eda726f38f4ddf0477e97bb79d URL: https://github.com/kamailio/kamailio/commit/556fab89cec8d1eda726f38f4ddf0477e97bb79d Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-05-15T16:40:28+03

[sr-dev] mtree module mt_match() api function

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

[sr-dev] presence: subsription double delete fix

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

[sr-dev] new compiler warning

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]

Re: [sr-dev] new compiler warning

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

[sr-dev] on reducing the number of email messages

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

[sr-dev] contact uri is too long

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

Re: [sr-dev] contact uri is too long

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

[sr-dev] contact uri is too long

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:

[sr-dev] contact uri is too long

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

[sr-dev] git:master:62ea33a6: increased location table contact field size from 255 to 512

Module: kamailio Branch: master Commit: 62ea33a6e9585e25362bd669c2f1cd11321dc119 URL: https://github.com/kamailio/kamailio/commit/62ea33a6e9585e25362bd669c2f1cd11321dc119 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-09-14T14:44:41+03:0

[sr-dev] git:master:5a1555ca: Merge branch 'master' of github.com:kamailio/kamailio

Module: kamailio Branch: master Commit: 5a1555ca8d52b112b4d4671583aeb0ed44422016 URL: https://github.com/kamailio/kamailio/commit/5a1555ca8d52b112b4d4671583aeb0ed44422016 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-09-14T14:46:23+

[sr-dev] got these docbook errors today

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:

[sr-dev] sipdump comment/question

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

[sr-dev] git:master:02074c81: Merge branch 'master' of github.com:kamailio/kamailio

Module: kamailio Branch: master Commit: 02074c81158b0927778e5f2daaae8da953580f2f URL: https://github.com/kamailio/kamailio/commit/02074c81158b0927778e5f2daaae8da953580f2f Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-10-16T12:41:48+

[sr-dev] git:master:8596e84b: sipdump: enable param now defaults to 0

Module: kamailio Branch: master Commit: 8596e84b20f93cf73963b0c61e15dbb049b2276a URL: https://github.com/kamailio/kamailio/commit/8596e84b20f93cf73963b0c61e15dbb049b2276a Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-10-16T12:40:43+03

[sr-dev] sipdump compile trouble

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

Re: [sr-dev] git:master:02074c81: Merge branch 'master' of github.com:kamailio/kamailio

Victor Seva writes: > rebase before pushing This is what I did (taken from shell history): 2060 emacs sipdump_mod.c 2061 git diff 2062 git commit -a 2063 git push 2064 git pull 2065 git push If I remember correctly, first git push failed with message asking to do pull first, which

Re: [sr-dev] git:master:02074c81: Merge branch 'master' of github.com:kamailio/kamailio

Victor Seva writes: > rebase before pushing Hard to remember. Is it possible to configure git so that it cancels push if local repo is not up to date? And even if had remembered to rebase, there would still be a cap between push during which a new commit could have occurred. Sound broken

Re: [sr-dev] git:master:02074c81: Merge branch 'master' of github.com:kamailio/kamailio

Victor Seva writes: > git pull --rebase is the missing step there. I have set pull rebase in > ~/.gitconfig [0] > > [pull] > rebase = true > > so git pull --rebase is automatic for me Thanks, Juha ___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:98609436: sipdump: implemented sipdump.enable rpc command

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

Re: [sr-dev] git:master:98609436: sipdump: implemented sipdump.enable rpc command

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

[sr-dev] git:master:06a5bb4c: modules/lcr: remove extra prefix_column param

Module: kamailio Branch: master Commit: 06a5bb4cff0319a19660b711e7d987b30a939345 URL: https://github.com/kamailio/kamailio/commit/06a5bb4cff0319a19660b711e7d987b30a939345 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-09-11T22:21:26+03

[sr-dev] git:master:b0f941e1: modules/lcr: improved description of prefix_column param

Module: kamailio Branch: master Commit: b0f941e1ef9a013472ea42bade3c8ecae9111768 URL: https://github.com/kamailio/kamailio/commit/b0f941e1ef9a013472ea42bade3c8ecae9111768 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-09-06T21:34:56+03

[sr-dev] git:master:b325a771: modules/lcr: fixed testing if dont_strip_or_prefix_flag is set

Module: kamailio Branch: master Commit: b325a7711ac405e5598f7fbd224fb1b8e84221cf URL: https://github.com/kamailio/kamailio/commit/b325a7711ac405e5598f7fbd224fb1b8e84221cf Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-08-25T10:29:44+03

[sr-dev] ul.add params

I noticed that my ul.add command that works OK in 5.0 doesn't anymore work in 5.1. I get back to the xmlrpc query: faultString Not enough parameters or wrong format I then went and compared ul.add in 5.0 and 5.1 and noticed that in 5.1 it has added two optional params: received

Re: [sr-dev] ul.add params

Juha Heinanen writes: > Perhaps the error is due to this change: > > in 5.0: >path value of the contact > > in 5.1: > path value with the Path vector (use '0' or '.' if it should not be set) > > In my xmlrpc command path p

Re: [sr-dev] ul.add params

Daniel-Constantin Mierla writes: > if you don't need to provide any of those parameters, then it should just > work by not having them in the command. OK, but it doesn't as the error message told. I should not have made any changes to that part of code from 5.0 to 5.1, but can recheck. -- Juha

Re: [sr-dev] ul.add params

Juha Heinanen writes: > > if you don't need to provide any of those parameters, then it should just > > work by not having them in the command. Perhaps the error is due to this change: in 5.0: path value of the contact in 5.1: path value with the Path vector (use '0' or '.'

Re: [sr-dev] ul.add params

Daniel-Constantin Mierla writes: > I will look at it. If you can the log messages with debug=3 in the config > for processing the command, I will try to spot it quicly. Otherwise likely > to take 1-2 days until I get back to office and can reproduce myself. I have been busy, but tries by adding

Re: [sr-dev] ul.add params

Daniel-Constantin Mierla writes: > It proved that the issue was in the rpc module implementation related to > handling of the optional specifier '*' when there were no more params > available. I pushed some patches to both jsonrpcs and xmlrpc, but I > could test only with jsonrpcs. Maybe you can

[sr-dev] add_contact_alias(0) is proto differs

When adding transport proto check to add_contact_alias, I noticed that in receive_info struct proto is defined like this: char proto; and in sip_uri struct like this: unsigned short proto; /*!< from transport */ That does not look good to me. They should have the same type in

[sr-dev] git:5.1:a7049edf: modules/nathelper: check also transport protocol when determining

Module: kamailio Branch: 5.1 Commit: a7049edf431b8b8f8c80040b3b005c2e00d44eca URL: https://github.com/kamailio/kamailio/commit/a7049edf431b8b8f8c80040b3b005c2e00d44eca Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-11-24T10:51:04+02

[sr-dev] git:5.0:ae575733: modules/nathelper: check also transport protocol when determining

Module: kamailio Branch: 5.0 Commit: ae575733a2761dba1232166f105eeb82781229e6 URL: https://github.com/kamailio/kamailio/commit/ae575733a2761dba1232166f105eeb82781229e6 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-11-24T10:58:57+02

Re: [sr-dev] rfc: clang format the code

Am Donnerstag, 23. November 2017, 17:20:48 CET schrieb Daniel-Constantin Mierla: > it was discussed during the last irc devel meeting and everyone there > agreed to use clang-format to format the source code: > > - https://clang.llvm.org/docs/ClangFormat.html Does anyone have emacs

[sr-dev] git:master:93c81f53: modules/nathelper: check also transport protocol when determining

Module: kamailio Branch: master Commit: 93c81f5326acc2aefaca01c347f79c972727d0bf URL: https://github.com/kamailio/kamailio/commit/93c81f5326acc2aefaca01c347f79c972727d0bf Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-11-24T10:18:33+02

[sr-dev] .clang-format error on debian stretch

clang-format command on debian stretch cannot hand kamailio .clang-format without errors. first one is this: YAML:58:21: error: unknown key 'IncludeIsMainRegex' IncludeIsMainRegex: '$' second is this: YAML:61:19: error: unknown key 'JavaScriptQuotes' JavaScriptQuotes: Leave is it OK if I

[sr-dev] git:master:f8d0a395: .clang-format: do not use bleeding edge keys

Module: kamailio Branch: master Commit: f8d0a395074cd066a9f3df26debf14cf26e21a12 URL: https://github.com/kamailio/kamailio/commit/f8d0a395074cd066a9f3df26debf14cf26e21a12 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-11-28T00:22:36+0

Re: [sr-dev] add_contact_alias(0) is proto differs

Daniel-Constantin Mierla writes: > I am fine to get such fields to the same type, just be careful to have > them aligned to 32bits from the start of the structure. I'm not qualified enough to do that kind change, since it may affect many parts of the core and many modules. -- Juha

[sr-dev] add_contact_alias(0) is proto differs

Currently add_contact_alias() works like this: Adds an “;alias=ip~port~transport” parameter to the contact URI containing either received ip, port, and transport protocol or those given as parameters. If called without parameters, “;alias” parameter is only added if received ip and port

[sr-dev] acc to db has stopped working

I noticed that accounting to database has stopped working. I don't know when it has happened, but at least I have the problem with master. My acc config is like this: modparam("acc", "db_url", "mysql://user:pass@127.0.0.1/sip_proxy_usage") modparam("acc", "db_flag", 7) modparam("acc",

[sr-dev] acc to db has stopped working

Found the reason why db acc had stopped working: I had removed async_workers parameter. Looks like acc module does not insert anything if db_insert_mode=2 and there is no async workers. From README, I get the impression that it should be doing standard insert in that case: 6.28.

[sr-dev] Multiple memleaks and segfauts in Kamailio v5.0.4

M S writes: > 1. Calling sip_trace method without any arguements or with only one > arguement (duplicate uri) crashes kamailio with segfault. My K does not crash when I make call like this: setflag(TRACE); sip_trace(); but I have not defined as many vars as you, but only these: # --

[sr-dev] [kamailio/kamailio] lcr: export rule_id_avp with matching rule id (#1546)

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

[sr-dev] MAX_FLAG

I'm running out of flags. Is there some good reason why flag_t is unsigned int instead of unsigned long? -- Juha ___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:22e4167e: modules: readme files regenerated - tcpops ... [skip ci]

Kamailio Dev writes: > +3.7. tcp_get_conid(hostport, pvname) > + > + Get the connection id based on target host:port. The connection id is > + set in the variable named by pvname parameter. So this means that there can never be more than one simultaneous tcp connection to a peer? -- Juha

[sr-dev] git:master:f7347016: lcr: fixed checking if there is more uris available for next_gw

Module: kamailio Branch: master Commit: f73470163b081f5c877dddff9c08e94073b0cc4b URL: https://github.com/kamailio/kamailio/commit/f73470163b081f5c877dddff9c08e94073b0cc4b Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2018-02-05T21:58:0

[sr-dev] git:5.1:732a3153: lcr: fixed checking if there is more uris available for next_gw

Module: kamailio Branch: 5.1 Commit: 732a3153a0a41d5f951fff85de607f0b46ae73da URL: https://github.com/kamailio/kamailio/commit/732a3153a0a41d5f951fff85de607f0b46ae73da Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2018-02-05T22:05:58+02:00

[sr-dev] compiler warnings in master's siptrace

CC (gcc) [M siptrace.so]siptrace.o siptrace.c: In function 'trace_onreply_out': siptrace.c:1227:6: warning: unused variable 'len' [-Wunused-variable] int len; ^~~ siptrace.c: In function 'siptrace_net_data_recv': siptrace.c:1477:1: warning: label 'error' defined but not

[sr-dev] dmq server address question

I tried to configure dmq module according to README, but got this kind of error: Feb 20 07:32:55 rautu /usr/bin/pres-serv[17983]: ERROR: dmq [dmq.c:228]: mod_init(): server_uri is not a socket the proxy is listening on In the config I have: modparam("dmq", "server_address",

Re: [sr-dev] dmq server address question

Charles Chance writes: > You’re not missing anything. DMQ only works with UDP. So no security over the Internet if TLS is not available? IPSec doesn't sound like a good idea. -- Juha ___ Kamailio (SER) - Development Mailing List

[sr-dev] git:5.1:edbfa1ea: usrloc: partion column name can be set via modparam

Øyvind Kolbu writes: > Module: kamailio > Branch: 5.1 > Commit: edbfa1eae2df1b932dff0bf5b7182ef09592a37d > URL: > https://github.com/kamailio/kamailio/commit/edbfa1eae2df1b932dff0bf5b7182ef09592a37d > > Author: Oyvind Kolbu > Committer: Øyvind Kolbu

Re: [sr-dev] new compiler warnings in master

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] new compiler warnings in master

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 = >is_null; ^ my_cmd.c: In function 'bind_result': my_cmd.c:1004:21: warning:

Re: [sr-dev] error in Makefile.modules

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

Re: [sr-dev] error in Makefile.modules

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/enti

[sr-dev] error in Makefile.modules

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

[sr-dev] git:master:26449dc2: modules/auth: fixed path in auth.xml

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

[sr-dev] jsonrpc reply error

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]:

Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

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

[sr-dev] [SR-Users] is t_flush_flags() really needed?

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

Re: [sr-dev] [SR-Users] is t_flush_flags() really needed?

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

Re: [sr-dev] no luck with xflags

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

[sr-dev] no luck with xflags

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

[sr-dev] git:master:b4aef50e: rtp_media_server: initial creation of README file

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

Re: [sr-dev] git:master:b4aef50e: rtp_media_server: initial creation of README file

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

Re: [sr-dev] git:master:b4aef50e: rtp_media_server: initial creation of README file

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

[sr-dev] git:master:5d0494f9: sl: new pv $ltt(key) - return local generated to tag

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

Re: [sr-dev] compiler warning

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

Re: [sr-dev] module related crash

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

Re: [sr-dev] module related crash

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

Re: [sr-dev] module related crash

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

Re: [sr-dev] module related crash

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-Constanti

Re: [sr-dev] module related crash

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)

Re: [sr-dev] module related crash

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

Re: [sr-dev] module related crash

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

[sr-dev] git:master:dd9615e7: modules/lcr: tried to test unsigned int values against both DB1_INT

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

[sr-dev] module related crash

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

Re: [sr-dev] removal of unused modules

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

Re: [sr-dev] compiler warning

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

[sr-dev] git:master:7e1112ab: modules/peering: added a note that the module is obsolete

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

[sr-dev] removal of unused modules

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

[sr-dev] uac module unused var in master

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] git:5.1:53bc3a93: core: backported commit f088d2afe4153b9e440a4293211c78f5a25af691 manually

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

Re: [sr-dev] [kamailio/kamailio] jsonrpc reply error (#1640)

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

[sr-dev] git:5.2:ca470cb6: pv: add $def pseudo variable

Many of today's commits by Sergey Safarov seem to add new features to stable release, for example: Module: kamailio Branch: 5.2 Commit: ca470cb6c5f341c518b208e8e595be2ed1309802 URL: https://github.com/kamailio/kamailio/commit/ca470cb6c5f341c518b208e8e595be2ed1309802 Author: lazedo

Re: [sr-dev] git:5.2:ca470cb6: pv: add $def pseudo variable

Daniel-Constantin Mierla writes: > They were pushed by mistake and they were reverted -- if you spot one > that wasn't, list it here and it will be reverted as well. OK, thanks, Juha ___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:95c18631: - added AUTH_USERNAME_EXPIRED auth api return code and used it in auth

Module: kamailio Branch: master Commit: 95c18631032c5510d4dc1e6b7aa815256ff3e688 URL: https://github.com/kamailio/kamailio/commit/95c18631032c5510d4dc1e6b7aa815256ff3e688 Author: Juha Heinanen Committer: Juha Heinanen Date: 2019-03-27T11:10:16+02:00 - added AUTH_USERNAME_EXPIRED auth api

[sr-dev] ephemeral auth username expired

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

Re: [sr-dev] still no fix to siptrace

Daniel-Constantin Mierla writes: > Anyhow, with Kamailio World finished, I just got a bit of time and > pushed a commit -- no time to test right now, but maybe you can give it > a try and then report if works. If not, I will dig further soon, just > give the new error messages/details/... There

Re: [sr-dev] [kamailio/kamailio] siptrace: fix tracing messages to database when destination uri not set (#1951)

Ionut Ionita writes: > Yes it works but the code in 5.2 is broken. Here's an example, maybe > it's more clear: > > sip_trace(); /* traces to database only no db url */ > sip_trace("", "$var(correlation_id)"); /* does not work; requires valid > duplicate uri */ > // with my additions >

  1   2   3   >