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

2018-11-13 Thread Henning Westerholt
Am Mittwoch, 14. November 2018, 07:00:08 CET schrieb Julien Chavanton: > Good question, not available at the moment since there is no call > transfer/bridge command yet. We always answer before we start the RTP > stream etc. The work is started to bridge call legs and I am hoping to > complete it s

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

2018-11-13 Thread Julien Chavanton
Good question, not available at the moment since there is no call transfer/bridge command yet. We always answer before we start the RTP stream etc. The work is started to bridge call legs and I am hoping to complete it soon. However, bridging early media may be suffisant, in the end "faking" a rem

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

2018-11-13 Thread Juha Heinanen
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

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

2018-11-13 Thread Juha Heinanen
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

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

2018-11-13 Thread Julien Chavanton
Hi Juha, I see the typo, I will correct it next time I improve the module. 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. Here is an example of the audio stream runnin

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

2018-11-13 Thread Juha Heinanen
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

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

2018-11-13 Thread Henning Westerholt
Module: kamailio Branch: master Commit: b4aef50ef2386e92c8c0bb1ad6232ae009d6621c URL: https://github.com/kamailio/kamailio/commit/b4aef50ef2386e92c8c0bb1ad6232ae009d6621c Author: Henning Westerholt Committer: Henning Westerholt Date: 2018-11-13T21:47:56+01:00 rtp_media_server: initial creation

Re: [sr-dev] [kamailio/kamailio] rtp_media_server: adding module (#1701)

2018-11-13 Thread Henning Westerholt
Thank you! -- 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/1701#issuecomment-438429536___ Kamailio (SER) - Development Mailing List sr-dev@

Re: [sr-dev] [kamailio/kamailio] 180 Ringing without Contact with TOPOS enabled (#1720)

2018-11-13 Thread Henning Westerholt
You are right, according to the RFC the Contact Header is optional in 1xx: https://tools.ietf.org/html/rfc3261#section-20.1 The RFC says as well that it SHOULD be included, but it is not a MUST. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or

Re: [sr-dev] [kamailio/kamailio] set_body_multipart produce broken body (#1719)

2018-11-13 Thread sergey-safarov
Thanks @sergey-vb I have done this and tested. Please looks messages 1) [config change](https://github.com/kamailio/kamailio/issues/1719#issuecomment-437619001); 2) [logs](https://github.com/kamailio/kamailio/issues/1719#issuecomment-437619183). -- You are receiving this because you are subsc

Re: [sr-dev] [kamailio/kamailio] set_body_multipart produce broken body (#1719)

2018-11-13 Thread Sergey Basov
> This may be related to log > > ``` > 16(27) ERROR: {1 21 INVITE 1851253425} [core/msg_translator.c:3290]: > sip_msg_apply_changes(): cannot apply msg changes after adding record-route > header - it breaks conditional 2nd header > ``` > > Think in this case must be send original INVITE. Hi.

[sr-dev] [kamailio/kamailio] 180 Ringing without Contact with TOPOS enabled (#1720)

2018-11-13 Thread Sergey Basov
### Description Kamailio with TOPOS enabled acts as a proxy between SIP client and SBC. When client sends "180 Ringing" without Contact header - kamailio does not forward it to SBC. If Contact header exists in "180 Ringing" - all works as expected. ### Troubleshooting Reproduction Load an

[sr-dev] git:master:1d89c0e5: Makefile.groups: added k group for rtp_media_server

2018-11-13 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 1d89c0e5310a4a0d64f9149f4131b45d001cea36 URL: https://github.com/kamailio/kamailio/commit/1d89c0e5310a4a0d64f9149f4131b45d001cea36 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-13T11:15:08+01:00 Makefile.groups: added

Re: [sr-dev] [kamailio/kamailio] rtp_media_server: adding module (#1701)

2018-11-13 Thread Julien Chavanton
Merged #1701 into master. -- 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/1701#event-1962350370___ Kamailio (SER) - Development Mailing Lis

[sr-dev] git:master:2a76fdd4: rtp_media_server: support for multiple actions

2018-11-13 Thread Julien Chavanton
Module: kamailio Branch: master Commit: 2a76fdd4336039da2df4db4459e6db4b5e3ba364 URL: https://github.com/kamailio/kamailio/commit/2a76fdd4336039da2df4db4459e6db4b5e3ba364 Author: Julien Chavanton Committer: Julien Chavanton Date: 2018-11-13T08:20:45Z rtp_media_server: support for multiple acti

[sr-dev] git:master:32c363e1: Merge pull request #1701 from jchavanton/rtp_media_server

2018-11-13 Thread GitHub
Module: kamailio Branch: master Commit: 32c363e1053d423bbc13c5d4e87a02097964ac7c URL: https://github.com/kamailio/kamailio/commit/32c363e1053d423bbc13c5d4e87a02097964ac7c Author: Julien Chavanton Committer: GitHub Date: 2018-11-13T00:34:58-08:00 Merge pull request #1701 from jchavanton/rtp_med

[sr-dev] git:master:47e35db0: rtp_media_server: adding module

2018-11-13 Thread Julien Chavanton
Module: kamailio Branch: master Commit: 47e35db05c5da1b2b2a24ae36b7b63d0625060c9 URL: https://github.com/kamailio/kamailio/commit/47e35db05c5da1b2b2a24ae36b7b63d0625060c9 Author: Julien Chavanton Committer: Julien Chavanton Date: 2018-11-13T07:23:46Z rtp_media_server: adding module --- Added

Re: [sr-dev] [kamailio/kamailio] rtp_media_server: adding module (#1701)

2018-11-13 Thread Julien Chavanton
I believe everything in the review was addressed I did remove the audio file and added a link to download one. I added another commit to support multiple actions this will be needed by some action like DTMF capture that could interrupt play and probably many other future commands. -- You are

[sr-dev] git:master:e2ca5516: tmx: use branch index for $T_rpl() cache invalidation

2018-11-13 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: e2ca5516ae086cf33c61dbb9e653a5484cca2074 URL: https://github.com/kamailio/kamailio/commit/e2ca5516ae086cf33c61dbb9e653a5484cca2074 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-13T08:59:43+01:00 tmx: use branch index