Re: [sr-dev] [kamailio/kamailio] Broken Kamailio repository for CentOS 7 (#1604)

2018-07-27 Thread Isaac Pohl-Zaretsky
+1

-- 
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/1604#issuecomment-408586109___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] topos: adding param contact_hostname (#1596)

2018-07-27 Thread Julien Chavanton
I think it is better to use the contact header param only taking optional port 
and transport from it.
And not use the Record route information at all this way we support NAPTR/SRV 
and things are less confusing for the user.

Thanks for the pointing this out, it was not my intent to use the port + 
transport from the RR.

I will : 
- update the PR and the doc too explicitly state that the host part can also 
contain params.
- parse the module contact param in mod_init and fail if is it invalid.
- make sure we are not using puri.port and puri.transport from the RR
- do a few tests

-- 
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/pull/1596#issuecomment-408566488___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] git:master:f8c46e63: modules: readme files regenerated - pipelimit ... [skip ci]

2018-07-27 Thread The Root
Module: kamailio
Branch: master
Commit: f8c46e6395ed9cc4cbcf062a823a9530bdec62c9
URL: 
https://github.com/kamailio/kamailio/commit/f8c46e6395ed9cc4cbcf062a823a9530bdec62c9

Author: Kamailio Dev 
Committer: The Root 
Date: 2018-07-27T22:17:10+02:00

modules: readme files regenerated - pipelimit ... [skip ci]

---

Modified: src/modules/pipelimit/README

---

Diff:  
https://github.com/kamailio/kamailio/commit/f8c46e6395ed9cc4cbcf062a823a9530bdec62c9.diff
Patch: 
https://github.com/kamailio/kamailio/commit/f8c46e6395ed9cc4cbcf062a823a9530bdec62c9.patch

---

diff --git a/src/modules/pipelimit/README b/src/modules/pipelimit/README
index 974eaed33b..3b48f2e0d6 100644
--- a/src/modules/pipelimit/README
+++ b/src/modules/pipelimit/README
@@ -51,12 +51,13 @@ Daniel-Constantin Mierla
 
 5. RPC Commands
 
-  5.1. pl.stats
-  5.2. pl.set_pipe
-  5.3. pl.get_pipes
-  5.4. pl.set_pid
-  5.5. pl.get_pid
-  5.6. pl.push_load
+  5.1. pl.list
+  5.2. pl.stats
+  5.3. pl.set_pipe
+  5.4. pl.get_pipes
+  5.5. pl.set_pid
+  5.6. pl.get_pid
+  5.7. pl.push_load
 
List of Examples
 
@@ -106,12 +107,13 @@ Chapter 1. Admin Guide
 
5. RPC Commands
 
-5.1. pl.stats
-5.2. pl.set_pipe
-5.3. pl.get_pipes
-5.4. pl.set_pid
-5.5. pl.get_pid
-5.6. pl.push_load
+5.1. pl.list
+5.2. pl.stats
+5.3. pl.set_pipe
+5.4. pl.get_pipes
+5.5. pl.set_pid
+5.6. pl.get_pid
+5.7. pl.push_load
 
 1. Overview
 
@@ -441,14 +443,30 @@ with unexpected retcode=$var(check_result)\n");
 
 5. RPC Commands
 
-   5.1. pl.stats
-   5.2. pl.set_pipe
-   5.3. pl.get_pipes
-   5.4. pl.set_pid
-   5.5. pl.get_pid
-   5.6. pl.push_load
+   5.1. pl.list
+   5.2. pl.stats
+   5.3. pl.set_pipe
+   5.4. pl.get_pipes
+   5.5. pl.set_pid
+   5.6. pl.get_pid
+   5.7. pl.push_load
 
-5.1.  pl.stats
+5.1.  pl.list
+
+   Lists the details of one or all pipes, respectively the attributes pipe
+   name (id), algorithm, limit and counter.
+
+   Name: pl.list
+
+   Parameters: name - (optional) pipe name
+
+   RPC Command Format:
+...
+kamctl rpc pl.list
+kamctl rpc pl.list testid
+...
+
+5.2.  pl.stats
 
Lists the parameters and variables in the pipelimit module: pipe id,
pipe load and pipe counter.
@@ -458,9 +476,11 @@ with unexpected retcode=$var(check_result)\n");
Parameters: none
 
RPC Command Format:
-kamcmd pl.stats
+...
+kamcmd pl.stats
+...
 
-5.2.  pl.set_pipe
+5.3.  pl.set_pipe
 
Sets the pipe parameters for the given pipe id.
 
@@ -472,9 +492,11 @@ with unexpected retcode=$var(check_result)\n");
  * pipe_limit - the limit assigned to the given pipe id.
 
RPC Command Format:
-kamcmd pl.set_pipe 2 RED 10
+...
+kamcmd pl.set_pipe 2 RED 10
+...
 
-5.3.  pl.get_pipes
+5.4.  pl.get_pipes
 
Gets the list of in use pipes.
 
@@ -483,9 +505,11 @@ with unexpected retcode=$var(check_result)\n");
Parameters: none
 
RPC Command Format:
-kamcmd pl.get_pipes
+...
+kamcmd pl.get_pipes
+...
 
-5.4.  pl.set_pid
+5.5.  pl.set_pid
 
Sets the PID Controller parameters for the Feedback Algorithm.
 
@@ -497,9 +521,11 @@ with unexpected retcode=$var(check_result)\n");
  * kd - the derivative parameter.
 
RPC Command Format:
-kamcmd pl.set_pid 0.5 0.5 0.5
+...
+kamcmd pl.set_pid 0.5 0.5 0.5
+...
 
-5.5.  pl.get_pid
+5.6.  pl.get_pid
 
Gets the list of in use PID Controller parameters.
 
@@ -508,9 +534,11 @@ with unexpected retcode=$var(check_result)\n");
Parameters: none
 
RPC Command Format:
-kamcmd pl.get_pid
+...
+kamcmd pl.get_pid
+...
 
-5.6.  pl.push_load
+5.7.  pl.push_load
 
Force the value of the load parameter. This command is useful for
testing the Feedback algorithm.
@@ -522,4 +550,6 @@ with unexpected retcode=$var(check_result)\n");
smaller then 1.0).
 
RPC Command Format:
-kamcmd pl.push_load 0.85
+...
+kamcmd pl.push_load 0.85
+...


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] git:master:674ab074: modules: readme files regenerated - uac ... [skip ci]

2018-07-27 Thread Kamailio Dev
Module: kamailio
Branch: master
Commit: 674ab0749960398fdddfde3772376b1b2461c107
URL: 
https://github.com/kamailio/kamailio/commit/674ab0749960398fdddfde3772376b1b2461c107

Author: Kamailio Dev 
Committer: Kamailio Dev 
Date: 2018-07-27T22:17:20+02:00

modules: readme files regenerated - uac ... [skip ci]

---

Modified: src/modules/uac/README

---

Diff:  
https://github.com/kamailio/kamailio/commit/674ab0749960398fdddfde3772376b1b2461c107.diff
Patch: 
https://github.com/kamailio/kamailio/commit/674ab0749960398fdddfde3772376b1b2461c107.patch

---

diff --git a/src/modules/uac/README b/src/modules/uac/README
index dae255ee37..b833372b6a 100644
--- a/src/modules/uac/README
+++ b/src/modules/uac/README
@@ -840,8 +840,11 @@ failure_route[REMOTE_AUTH] {
 
 6.1.  event_route[uac:reply]
 
-   Event route executed for the final reply to the request set with
-   uac_req_send(). The associated $uac_req(evroute) has to be set to 1.
+   Event route executed for the final reply of the branch for the request
+   sent with uac_req_send(). The associated $uac_req(evroute) has to be
+   set to 1. If the request is challenged for authentication with 401/407,
+   then the event_route is executed twice, first for 401/407 and second
+   for final reply of the transaction.
 
Example 1.33. event_route[uac:reply] usage
 ...


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] git:master:1cec5e80: uac: docs - more details about execution of event_route[uac:reply]

2018-07-27 Thread Daniel-Constantin Mierla
Module: kamailio
Branch: master
Commit: 1cec5e80036808badf19c8c72ec243b504f56e28
URL: 
https://github.com/kamailio/kamailio/commit/1cec5e80036808badf19c8c72ec243b504f56e28

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-07-27T22:00:47+02:00

uac: docs - more details about execution of event_route[uac:reply]

---

Modified: src/modules/uac/doc/uac_admin.xml

---

Diff:  
https://github.com/kamailio/kamailio/commit/1cec5e80036808badf19c8c72ec243b504f56e28.diff
Patch: 
https://github.com/kamailio/kamailio/commit/1cec5e80036808badf19c8c72ec243b504f56e28.patch

---

diff --git a/src/modules/uac/doc/uac_admin.xml 
b/src/modules/uac/doc/uac_admin.xml
index 162f491b31..8047c67c2b 100644
--- a/src/modules/uac/doc/uac_admin.xml
+++ b/src/modules/uac/doc/uac_admin.xml
@@ -985,9 +985,11 @@ failure_route[REMOTE_AUTH] {
event_route[uac:reply]


-   Event route executed for the final reply to the request 
set with
-   uac_req_send(). The associated $uac_req(evroute) has to 
be set to
-   1.
+   Event route executed for the final reply of the branch 
for the
+   request sent with uac_req_send(). The associated 
$uac_req(evroute)
+   has to be set to 1. If the request is challenged for 
authentication
+   with 401/407, then the event_route is executed twice, 
first for
+   401/407 and second for final reply of the transaction.


event_route[uac:reply] usage


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] core/parser:parse_headers error. (#1582)

2018-07-27 Thread Daniel-Constantin Mierla
Any time to grab that pcap?

-- 
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/1582#issuecomment-408517505___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] topos_redis not included in debian packages for kamailio-redis-modules (#1603)

2018-07-27 Thread Victor Seva
>From 
>[kamailio-5.1.4](https://kamailio.sipwise.com/view/kam51/job/kamailio51-binaries/architecture=amd64,distribution=stretch,label=slave/6/consoleText)
> build

```
kamailio-redis-modules_5.1.4+stretch_amd64.deb
--
 new debian package, version 2.0.
 size 243754 bytes: control archive=849 bytes.
 662 bytes,17 lines  control  
 605 bytes, 7 lines  md5sums  
 Package: kamailio-redis-modules
 Source: kamailio
 Version: 5.1.4+stretch
 Architecture: amd64
 Maintainer: Debian VoIP Team 
 Installed-Size: 560
 Depends: kamailio (= 5.1.4+stretch), libc6 (>= 2.14), libhiredis0.13 (>= 
0.13.1)
 Section: net
 Priority: optional
 Multi-Arch: same
 Homepage: http://www.kamailio.org/
 Description: Redis database connectivity module for the Kamailio SIP server
  Kamailio is a very fast and flexible SIP (RFC3261)
  server. Written entirely in C, Kamailio can handle thousands calls
  per second even on low-budget hardware.
  .
  This package provides the Redis NOSQL database driver for Kamailio.
drwxr-xr-x root/root 0 2018-06-05 14:20 ./
drwxr-xr-x root/root 0 2018-06-05 14:20 ./usr/
drwxr-xr-x root/root 0 2018-06-05 14:20 ./usr/lib/
drwxr-xr-x root/root 0 2018-06-05 14:20 ./usr/lib/x86_64-linux-gnu/
drwxr-xr-x root/root 0 2018-06-05 14:20 
./usr/lib/x86_64-linux-gnu/kamailio/
drwxr-xr-x root/root 0 2018-06-05 14:20 
./usr/lib/x86_64-linux-gnu/kamailio/modules/
-rw-r--r-- root/root192504 2018-06-05 14:20 
./usr/lib/x86_64-linux-gnu/kamailio/modules/ndb_redis.so
-rw-r--r-- root/root183344 2018-06-05 14:20 
./usr/lib/x86_64-linux-gnu/kamailio/modules/topos_redis.so
drwxr-xr-x root/root 0 2018-06-05 14:20 ./usr/share/
drwxr-xr-x root/root 0 2018-06-05 14:20 ./usr/share/doc/
drwxr-xr-x root/root 0 2018-06-05 14:20 ./usr/share/doc/kamailio/
drwxr-xr-x root/root 0 2018-06-05 14:20 
./usr/share/doc/kamailio/modules/
-rw-r--r-- root/root  4207 2018-06-05 14:20 
./usr/share/doc/kamailio/modules/README.ndb_redis.gz
-rw-r--r-- root/root  1940 2018-06-05 14:20 
./usr/share/doc/kamailio/modules/README.topos_redis
drwxr-xr-x root/root 0 2018-06-05 14:20 
./usr/share/doc/kamailio-redis-modules/
-rw-r--r-- root/root  1982 2018-06-05 14:20 
./usr/share/doc/kamailio-redis-modules/changelog.Debian.gz
-rw-r--r-- root/root158404 2018-06-05 14:20 
./usr/share/doc/kamailio-redis-modules/changelog.gz
-rw-r--r-- root/root 16208 2017-12-11 16:44 
./usr/share/doc/kamailio-redis-modules/copyright
```

-- 
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/1603#issuecomment-408455266___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] topos_redis not included in debian packages for kamailio-redis-modules (#1603)

2018-07-27 Thread Victor Seva
Closed #1603.

-- 
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/1603#event-1757555094___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] Fix no cdr after restart (#1606)

2018-07-27 Thread Daniel-Constantin Mierla
Thank for digging into it and coming with a fix.
Can you reformat the commit message so it matches the contributions guide:

  * 
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format

So, the it should be like:

```
dialog: fix to get CDRs after restart

[more details here]
```

If you don't get the chance to do it soon, no worries, I can apply the patch 
manually, just have it in mind for future pull requests. It makes life easier 
by merging with one click here, instead of editing/patching source manually.

-- 
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/pull/1606#issuecomment-408410640___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] cfg_rpc updates changing variable values to large random numbers (#1583)

2018-07-27 Thread Daniel-Constantin Mierla
I was traveling (actually still there) in USA (Cluecon), so I didn't get a 
chance to try to reproduce, hopefully soon it will happen.

-- 
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/1583#issuecomment-408409657___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] MemoryUtilization getting increase with lua routing (#1577)

2018-07-27 Thread Daniel-Constantin Mierla
Didn't get the chance to look at it, being in a trip to USA (Cluecon). Hope to 
get back to it soon, after returning.

-- 
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/1577#issuecomment-408409299___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] topos_redis not included in debian packages for kamailio-redis-modules (#1603)

2018-07-27 Thread Daniel-Constantin Mierla
I am not sure if that `apt-file search` goes inside debs, but `topos_redis` 
should be part of `kamailio-redis-modules` deb. If not there, then needs to be 
looked more at what happened at build time.

-- 
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/1603#issuecomment-408409029___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] topos: adding param contact_hostname (#1596)

2018-07-27 Thread Daniel-Constantin Mierla
Just to clarify: do you want that the port and transport from Record-Route to 
be kept in the Contact when using the parameter with the domain, right? That 
practically results only in doing an A/AAA DNS query on the domain, if you want 
SRV or NAPTR to be done, then this way is not working, but I see also the 
benefits of preserving the port and transport from R-R. 

-- 
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/pull/1596#issuecomment-408407922___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] topos is masking the contact in 302 redirect response from outside

2018-07-27 Thread Daniel-Constantin Mierla
Hello,

thanks for testing and feedback -- likely in two weeks or so we should
do a new release in 5.1 branch and this will get in.

Cheers,
Daniel


On 27.07.18 08:02, Andrew Pogrebennyk wrote:
> Hi Daniel,
> it works for me! Thank you for the quick fix.
>
> Regards,
> Andrew
>
> Daniel-Constantin Mierla wrote:
>> Hello,
>>
>>
>> On 24.07.18 15:41, Andrew Pogrebennyk wrote:
>>
>> - hide quoted text -- show quoted text -
 Hello,
 using kamailio 5.1.4: the topos is changing the Contact in 302 response
 from the UA. Has anyone else observed it so it's a known issue or should
 I file a proper bugreport? At a glance, I don't see any related fixes in
 master, so i guess i will do it anyway this week.. In meantime I'll try
 to workaround it for my purpose using new even route [topos:msg-sending]
 Thanks in advance for your hints.


>> the scenario for 3xx was not addressed so far in the way of keeping
>> incoming Contact addresses. I just pushed a patch for it, can you give a
>> try and report if works as expected or not?
>>
>> Commit:
>> https://github.com/kamailio/kamailio/commit/fcf1d3d778a903852b6e21103a23bd7fd6e3bbc9
>>
>> Cheers,
>> Daniel

-- 
Daniel-Constantin Mierla -- www.asipto.com
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference -- www.kamailioworld.com


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] Update dlg_cb.c (#1607)

2018-07-27 Thread kosza




 Pre-Submission Checklist



- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, 
...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)

 Checklist:

- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue # (replace  with an open issue number)

 Description


The load_cbs and create_cbs were interchanged in line 136.
You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1607

-- Commit Summary --

  * Update dlg_cb.c

-- File Changes --

M src/modules/dialog/dlg_cb.c (2)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1607.patch
https://github.com/kamailio/kamailio/pull/1607.diff

-- 
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/pull/1607
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] Fix no cdr after restart (#1606)

2018-07-27 Thread kosza
Hello!

I am using kamailio 5.1.4.
version: kamailio 5.1.4 (x86_64/linux)
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, DISABLE_NAGLE, 
USE_MCAST, DNS_IP_HACK, SHM_MEM, 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_LISTEN 16, 
MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: unknown
compiled on 15:11:36 Jul 20 2018 with gcc 7.3.0

The problem is that if kamailio is restarted during a call the cdr would not be 
created. 

The run_load_callbacks() should be call after acc module is loaded.
The acc module will setup DLGCB_LOADED type callback. So run_load_callbacks() 
should be delayed and run only once, the child_init(PROC_MAIN) is perfect for 
it.





 Pre-Submission Checklist



- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, 
...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)

 Checklist:

- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue # (replace  with an open issue number)

 Description


You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1606

-- Commit Summary --

  * Fix no cdr after restart bug

-- File Changes --

M src/modules/dialog/dialog.c (5)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1606.patch
https://github.com/kamailio/kamailio/pull/1606.diff

-- 
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/pull/1606
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] ims_ipsec_pcscf: sec-agree implementation for IMS (#1605)

2018-07-27 Thread Tsvetomir Dimitrov




 Pre-Submission Checklist



- [X] Commit message has the format required by CONTRIBUTING guide
- [X] Commits are split per component (core, individual modules, libs, utils, 
...)
- [X] Each component has a single commit (if not, squash them into one commit)
- [X] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [X] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)

 Checklist:

- [ ] PR should be backported to stable branches
- [X] Tested changes locally
- [ ] Related to issue # (replace  with an open issue number)

 Description


This is an implementation of sec-agree used in IMS with IPSec. It's not a 
complete sec-agree implementation, only the flows used by IMS. The code is 
usable, but needs a few improvements, which I plan to push in the near future. 
**My work is based on the implementation in OpenIMSCore.**

For IPSec implementation the XFRM framework from the Linux kernel is used. 
Security association (SA) and Policies creation/removal is performed via 
netlink messages. For this reason the module depends on libmnl (a minimalistic 
netlink library).

As XFRM is Linux specific, the code is not portable and can't be used on 
operating system different from Linux. The code will not compile on *BSDs too. 
However all platform specific code resides in ipsec.c so support for other 
OSes/IPSec implementations can be added relatively easy.

The README file, which is commited is generated from docs dir with xsltproc.

Issues I still work on:
- Kamailio must be run as root in order to be able to send netlink messages and 
create XFRM SAs and Policies.
- SAs and Policies are not deleted on Kamailio startup and shutdown.
- According to the current contact implementation in the PCSCF modules (adn the 
3GPP specs) the IPSec tunnel should be created on two steps. Initial parameters 
should be saved in security_tmp and on confirmation - in security. At the 
moment everything remains in security.

As this is my first more serious contribution to the project, all kinds of 
feedback is highly appreciated :)
You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1605

-- Commit Summary --

  * ims_ipsec_pcscf: sec-agree implementation for IMS

-- File Changes --

A src/modules/ims_ipsec_pcscf/Makefile (20)
A src/modules/ims_ipsec_pcscf/README (244)
A src/modules/ims_ipsec_pcscf/cmd.c (585)
A src/modules/ims_ipsec_pcscf/cmd.h (6)
A src/modules/ims_ipsec_pcscf/doc/Makefile (4)
A src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf.xml (88)
A src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml (231)
A src/modules/ims_ipsec_pcscf/ims_ipsec_pcscf_mod.c (227)
A src/modules/ims_ipsec_pcscf/ipsec.c (386)
A src/modules/ims_ipsec_pcscf/ipsec.h (24)
A src/modules/ims_ipsec_pcscf/run_spi_list_tests.sh (4)
A src/modules/ims_ipsec_pcscf/spi_gen.c (87)
A src/modules/ims_ipsec_pcscf/spi_gen.h (16)
A src/modules/ims_ipsec_pcscf/spi_list.c (123)
A src/modules/ims_ipsec_pcscf/spi_list.h (29)
A src/modules/ims_ipsec_pcscf/spi_list_tests.c (292)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1605.patch
https://github.com/kamailio/kamailio/pull/1605.diff

-- 
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/pull/1605
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] MemoryUtilization getting increase with lua routing (#1577)

2018-07-27 Thread Surendra Tiwari
@miconda please let me know if this issue is reproduced by the same config at 
your side or not

-- 
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/1577#issuecomment-408352787___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev