sr-dev May 2020

sr-dev@lists.kamailio.org

24 participants
393 discussions

[kamailio/kamailio] error: 'pkg/docker' does not have a commit checked out (#2003)
by Daniel-Constantin Mierla 19 May '20

19 May '20

When I just tried to do a commit today, I got the following error: ``` kamailio $ git ci . error: 'pkg/docker' does not have a commit checked out fatal: updating files failed ``` It is related to the `pkg/docker` submodule referencing to kamailio/kamailio-ci. Not sure if this is due to a new git version or something else, because I did commits in that folder a few days ago. To get is sorted out, I did: ``` kamailio $ git submodule update pkg/docker Submodule path 'pkg/docker' not initialized Maybe you want to use 'update --init'? kamailio $ git submodule update --init pkg/docker Submodule 'pkg/docker' (https://github.com/kamailio/kamailio-ci.git) registered for path 'pkg/docker' Cloning into '/.../kamailio/pkg/docker'... Submodule path 'pkg/docker': checked out 'b92a1c899b8c474afadd0cf29dd7cb4ea01a3466' ``` Then it worked to do the commit. I wonder if this is going to break the process we were used with, specially the installation guidelines from GIT repo. Therefore I open this item to see if we really need the to have `pkg/docker` submodule, or better have the docker files directly in the repo. Other options to keep things sane and simple? Git version: ``` $ git --version git version 2.22.0 ``` -- 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/2003

5 10

git:master:f2458a66: core: kemi - functions to test http methods
by Daniel-Constantin Mierla 19 May '20

19 May '20

Module: kamailio Branch: master Commit: f2458a66ae46a74b83fe2a3f3886f6015249a960 URL: https://github.com/kamailio/kamailio/commit/f2458a66ae46a74b83fe2a3f3886f60… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-19T14:23:01+02:00 core: kemi - functions to test http methods --- Modified: src/core/kemi.c --- Diff: https://github.com/kamailio/kamailio/commit/f2458a66ae46a74b83fe2a3f3886f60… Patch: https://github.com/kamailio/kamailio/commit/f2458a66ae46a74b83fe2a3f3886f60… --- diff --git a/src/core/kemi.c b/src/core/kemi.c index c15621690a..2a63279795 100644 --- a/src/core/kemi.c +++ b/src/core/kemi.c @@ -1076,6 +1076,38 @@ static int sr_kemi_core_is_method_kdmq(sip_msg_t *msg) } +/** + * + */ +static int sr_kemi_core_is_method_get(sip_msg_t *msg) +{ + return sr_kemi_core_is_method_type(msg, METHOD_GET); +} + +/** + * + */ +static int sr_kemi_core_is_method_post(sip_msg_t *msg) +{ + return sr_kemi_core_is_method_type(msg, METHOD_POST); +} + +/** + * + */ +static int sr_kemi_core_is_method_put(sip_msg_t *msg) +{ + return sr_kemi_core_is_method_type(msg, METHOD_PUT); +} + +/** + * + */ +static int sr_kemi_core_is_method_delete(sip_msg_t *msg) +{ + return sr_kemi_core_is_method_type(msg, METHOD_DELETE); +} + /** * */ @@ -1769,6 +1801,26 @@ static sr_kemi_t _sr_kemi_core[] = { { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, + { str_init(""), str_init("is_GET"), + SR_KEMIP_BOOL, sr_kemi_core_is_method_get, + { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + { str_init(""), str_init("is_POST"), + SR_KEMIP_BOOL, sr_kemi_core_is_method_post, + { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + { str_init(""), str_init("is_PUT"), + SR_KEMIP_BOOL, sr_kemi_core_is_method_put, + { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + { str_init(""), str_init("is_DELETE"), + SR_KEMIP_BOOL, sr_kemi_core_is_method_delete, + { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, { str_init(""), str_init("is_UDP"), SR_KEMIP_BOOL, sr_kemi_core_is_proto_udp, { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE,

1 0

git:master:d14833d9: dispatcher: sync on load management operations
by Daniel-Constantin Mierla 19 May '20

19 May '20

Module: kamailio Branch: master Commit: d14833d9d1ad6cc92c65e279b7dd89b3b922f129 URL: https://github.com/kamailio/kamailio/commit/d14833d9d1ad6cc92c65e279b7dd89b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-19T13:43:10+02:00 dispatcher: sync on load management operations - use locks to increment/decrement call load and get least loaded dst - GH #2322 --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/d14833d9d1ad6cc92c65e279b7dd89b… Patch: https://github.com/kamailio/kamailio/commit/d14833d9d1ad6cc92c65e279b7dd89b… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index 589b54f5b8..b51b5b48d0 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -79,6 +79,22 @@ #define DS_ALG_RELWEIGHT 11 #define DS_ALG_PARALLEL 12 +/* increment call load */ +#define DS_LOAD_INC(dgrp, didx) do { \ + lock_get(&(dgrp)->lock); \ + (dgrp)->dlist[didx].dload++; \ + lock_release(&(dgrp)->lock); \ + } while(0) + +/* decrement call load */ +#define DS_LOAD_DEC(dgrp, didx) do { \ + lock_get(&(dgrp)->lock); \ + if(likely((dgrp)->dlist[didx].dload > 0)) { \ + (dgrp)->dlist[didx].dload--; \ + } \ + lock_release(&(dgrp)->lock); \ + } while(0) + static int _ds_table_version = DS_TABLE_VERSION; static ds_ht_t *_dsht_load = NULL; @@ -1514,6 +1530,7 @@ int ds_get_leastloaded(ds_set_t *dset) k = -1; t = 0x7fffffff; /* high load */ + lock_get(&dset->lock); \ for(j = 0; j < dset->nr; j++) { if(!ds_skip_dst(dset->dlist[j].flags) && (dset->dlist[j].attrs.maxload == 0 @@ -1525,6 +1542,7 @@ int ds_get_leastloaded(ds_set_t *dset) } } } + lock_release(&dset->lock); \ return k; } @@ -1546,7 +1564,7 @@ int ds_load_add(struct sip_msg *msg, ds_set_t *dset, int setid, int dst) msg->callid->body.s); return -1; } - dset->dlist[dst].dload++; + DS_LOAD_INC(dset, dst); return 0; } @@ -1614,8 +1632,7 @@ int ds_load_replace(struct sip_msg *msg, str *duid) ds_unlock_cell(_dsht_load, &msg->callid->body); ds_del_cell(_dsht_load, &msg->callid->body); - if(idx->dlist[olddst].dload > 0) - idx->dlist[olddst].dload--; + DS_LOAD_DEC(idx, olddst); if(ds_load_add(msg, idx, set, newdst) < 0) { LM_ERR("unable to replace destination load [%.*s / %.*s]\n", duid->len, @@ -1654,8 +1671,7 @@ int ds_load_remove_byid(int set, str *duid) return -1; } - if(idx->dlist[olddst].dload > 0) - idx->dlist[olddst].dload--; + DS_LOAD_DEC(idx, olddst); return 0; }

1 0

[kamailio/kamailio] Siptrace Module is ignoring force_send_sock parameter (#2330)
by iliesh 19 May '20

19 May '20

### Description Trying to duplicate TLS SIP messages to the remote host directly from Kamailio by using Siptrace Module. Those are the parameters used by the module: ``` modparam("siptrace", "trace_on", 1) modparam("siptrace", "force_send_sock", "sip:192.168.1.1:5000") modparam("siptrace", "trace_mode", 0) modparam("siptrace", "trace_flag", 22) modparam("siptrace", "duplicate_uri", "sip:192.168.1.100") modparam("siptrace", "trace_to_database", 0) modparam("siptrace", "hep_mode_on", 0) modparam("siptrace", "xheaders_write", 1) modparam("siptrace", "trace_local_ip", "MY_EXTERNAL_IP_ADDR") modparam("siptrace", "trace_sl_acks", 0) ``` But seems that Kamailio is ignoring the parameter "force_send_sock" and keep sending from the first listen interface from the configuration file. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.2.7 (x86_64/linux) 9ea54f ``` * **Operating System**:  ``` CentsOS 7 3.10.0-1127.el7.x86_64 ``` -- 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/2330

2 2

[kamailio/kamailio] keepalive: early start of OPTIONS checking (#2326)
by Nacho G. 19 May '20

19 May '20

- Current code takes some time to start checking (ping_interval). So, if someone sets a really high ping_interval it takes a while to get destination status. This change fires the first OPTIONS check just 3 seconds (fixed) after the destination is added. The checks would be done with the pace defined in ping_interval.   #### 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 - [x] 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 - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  Current code takes some time to start checking (ping_interval). So, if someone sets a really high ping_interval it takes a while to get destination status. This change fires the first OPTIONS check just 3 seconds (fixed) after the destination is added. The checks would be done with the pace defined in ping_interval. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2326 -- Commit Summary -- * keepalive: early start of OPTIONS checking -- File Changes -- M src/modules/keepalive/keepalive.h (5) M src/modules/keepalive/keepalive_api.c (4) M src/modules/keepalive/keepalive_core.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2326.patch https://github.com/kamailio/kamailio/pull/2326.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/2326

2 4

git:master:b26951c0: Merge pull request #2326 from NGSegovia/keepalive/first_check_on_start
by GitHub 19 May '20

19 May '20

Module: kamailio Branch: master Commit: b26951c06483a0e9720b65689179541b29c938c5 URL: https://github.com/kamailio/kamailio/commit/b26951c06483a0e9720b65689179541… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2020-05-19T12:56:58+02:00 Merge pull request #2326 from NGSegovia/keepalive/first_check_on_start keepalive: early start of OPTIONS checking --- Modified: src/modules/keepalive/keepalive.h Modified: src/modules/keepalive/keepalive_api.c Modified: src/modules/keepalive/keepalive_core.c --- Diff: https://github.com/kamailio/kamailio/commit/b26951c06483a0e9720b65689179541… Patch: https://github.com/kamailio/kamailio/commit/b26951c06483a0e9720b65689179541… --- diff --git a/src/modules/keepalive/keepalive.h b/src/modules/keepalive/keepalive.h index a8da607db8..41e8eab00a 100644 --- a/src/modules/keepalive/keepalive.h +++ b/src/modules/keepalive/keepalive.h @@ -48,6 +48,8 @@ extern int ka_ping_interval; #define KA_PROBE_INACTIVE 2 #define KA_PROBE_ONLYFLAGGED 3 +#define KA_FIRST_TRY_DELAY 500 /* First OPTIONS send is done 500 millis after adding the destination */ + typedef void (*ka_statechanged_f)(str *uri, int state, void *user_attr); typedef void (*ka_response_f)( str *uri, struct tmcb_params *ps, void *user_attr); @@ -62,6 +64,7 @@ typedef struct _ka_dest int state; time_t last_checked, last_up, last_down; int counter; // counts unreachable attemps + ticks_t ping_interval; /*!< Actual interval between OPTIONS */ void *user_attr; ka_statechanged_f statechanged_clb; @@ -70,7 +73,7 @@ typedef struct _ka_dest struct ip_addr ip_address; /*!< IP-Address of the entry */ unsigned short int port; /*!< Port of the URI */ unsigned short int proto; /*!< Protocol of the URI */ - struct timer_ln *timer; + struct timer_ln *timer; /*!< Timer firing the OPTIONS test */ struct _ka_dest *next; } ka_dest_t; diff --git a/src/modules/keepalive/keepalive_api.c b/src/modules/keepalive/keepalive_api.c index 18336084af..19339c4ba6 100644 --- a/src/modules/keepalive/keepalive_api.c +++ b/src/modules/keepalive/keepalive_api.c @@ -107,6 +107,7 @@ int ka_add_dest(str *uri, str *owner, int flags, int ping_interval, dest->statechanged_clb = statechanged_clb; dest->response_clb = response_clb; dest->user_attr = user_attr; + dest->ping_interval = MS_TO_TICKS((ping_interval == 0 ? ka_ping_interval : ping_interval) * 1000) ; dest->timer = timer_alloc(); if (dest->timer == NULL) { @@ -116,8 +117,7 @@ int ka_add_dest(str *uri, str *owner, int flags, int ping_interval, timer_init(dest->timer, ka_check_timer, dest, 0); - int actual_ping_interval = ping_interval == 0 ? ka_ping_interval : ping_interval; - if(timer_add(dest->timer, MS_TO_TICKS(actual_ping_interval * 1000)) < 0){ + if(timer_add(dest->timer, MS_TO_TICKS(KA_FIRST_TRY_DELAY)) < 0){ LM_ERR("failed to start timer\n"); goto err; } diff --git a/src/modules/keepalive/keepalive_core.c b/src/modules/keepalive/keepalive_core.c index 04b520a62b..4daa57908d 100644 --- a/src/modules/keepalive/keepalive_core.c +++ b/src/modules/keepalive/keepalive_core.c @@ -82,7 +82,7 @@ ticks_t ka_check_timer(ticks_t ticks, struct timer_ln* tl, void* param) ka_dest->last_checked = time(NULL); - return (ticks_t)(-1); /* periodical */ + return ka_dest->ping_interval; /* periodical, but based on dest->ping_interval, not on initial_timeout */ } /*! \brief

1 0

git:master:f6b5f782: keepalive: early start of OPTIONS checking
by Nacho Garcia Segovia 19 May '20

19 May '20

Module: kamailio Branch: master Commit: f6b5f782e0ebc8570d640415c61b9bf654336154 URL: https://github.com/kamailio/kamailio/commit/f6b5f782e0ebc8570d640415c61b9bf… Author: Nacho Garcia Segovia <nacho.gs(a)zaleos.net> Committer: Nacho Garcia Segovia <nacho.gs(a)zaleos.net> Date: 2020-05-15T08:21:15+02:00 keepalive: early start of OPTIONS checking - Current code takes some time to start checking (ping_interval). So, if someone sets a really high ping_interval it takes a while to get destination status. This change fires the first OPTIONS check just 3 seconds (fixed) after the destination is added. The checks would be done with the pace defined in ping_interval. --- Modified: src/modules/keepalive/keepalive.h Modified: src/modules/keepalive/keepalive_api.c Modified: src/modules/keepalive/keepalive_core.c --- Diff: https://github.com/kamailio/kamailio/commit/f6b5f782e0ebc8570d640415c61b9bf… Patch: https://github.com/kamailio/kamailio/commit/f6b5f782e0ebc8570d640415c61b9bf… --- diff --git a/src/modules/keepalive/keepalive.h b/src/modules/keepalive/keepalive.h index a8da607db8..41e8eab00a 100644 --- a/src/modules/keepalive/keepalive.h +++ b/src/modules/keepalive/keepalive.h @@ -48,6 +48,8 @@ extern int ka_ping_interval; #define KA_PROBE_INACTIVE 2 #define KA_PROBE_ONLYFLAGGED 3 +#define KA_FIRST_TRY_DELAY 500 /* First OPTIONS send is done 500 millis after adding the destination */ + typedef void (*ka_statechanged_f)(str *uri, int state, void *user_attr); typedef void (*ka_response_f)( str *uri, struct tmcb_params *ps, void *user_attr); @@ -62,6 +64,7 @@ typedef struct _ka_dest int state; time_t last_checked, last_up, last_down; int counter; // counts unreachable attemps + ticks_t ping_interval; /*!< Actual interval between OPTIONS */ void *user_attr; ka_statechanged_f statechanged_clb; @@ -70,7 +73,7 @@ typedef struct _ka_dest struct ip_addr ip_address; /*!< IP-Address of the entry */ unsigned short int port; /*!< Port of the URI */ unsigned short int proto; /*!< Protocol of the URI */ - struct timer_ln *timer; + struct timer_ln *timer; /*!< Timer firing the OPTIONS test */ struct _ka_dest *next; } ka_dest_t; diff --git a/src/modules/keepalive/keepalive_api.c b/src/modules/keepalive/keepalive_api.c index 18336084af..19339c4ba6 100644 --- a/src/modules/keepalive/keepalive_api.c +++ b/src/modules/keepalive/keepalive_api.c @@ -107,6 +107,7 @@ int ka_add_dest(str *uri, str *owner, int flags, int ping_interval, dest->statechanged_clb = statechanged_clb; dest->response_clb = response_clb; dest->user_attr = user_attr; + dest->ping_interval = MS_TO_TICKS((ping_interval == 0 ? ka_ping_interval : ping_interval) * 1000) ; dest->timer = timer_alloc(); if (dest->timer == NULL) { @@ -116,8 +117,7 @@ int ka_add_dest(str *uri, str *owner, int flags, int ping_interval, timer_init(dest->timer, ka_check_timer, dest, 0); - int actual_ping_interval = ping_interval == 0 ? ka_ping_interval : ping_interval; - if(timer_add(dest->timer, MS_TO_TICKS(actual_ping_interval * 1000)) < 0){ + if(timer_add(dest->timer, MS_TO_TICKS(KA_FIRST_TRY_DELAY)) < 0){ LM_ERR("failed to start timer\n"); goto err; } diff --git a/src/modules/keepalive/keepalive_core.c b/src/modules/keepalive/keepalive_core.c index 04b520a62b..4daa57908d 100644 --- a/src/modules/keepalive/keepalive_core.c +++ b/src/modules/keepalive/keepalive_core.c @@ -82,7 +82,7 @@ ticks_t ka_check_timer(ticks_t ticks, struct timer_ln* tl, void* param) ka_dest->last_checked = time(NULL); - return (ticks_t)(-1); /* periodical */ + return ka_dest->ping_interval; /* periodical, but based on dest->ping_interval, not on initial_timeout */ } /*! \brief

1 0

[kamailio/kamailio] core: add case functions for str_hash get and set (#2332)
by Victor Seva 19 May '20

19 May '20

#### 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 #### Description Add str_hash_case_get() and str_hash_case_set() functions You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2332 -- Commit Summary -- * core: add case functions for str_hash get and set -- File Changes -- M src/core/str_hash.h (34) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2332.patch https://github.com/kamailio/kamailio/pull/2332.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/2332

2 2

git:master:6ffe43a5: core: add case functions for str_hash get and set
by Victor Seva 19 May '20

19 May '20

Module: kamailio Branch: master Commit: 6ffe43a50cbc33bb83b14f3aef3341af6b34e6f6 URL: https://github.com/kamailio/kamailio/commit/6ffe43a50cbc33bb83b14f3aef3341a… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-05-19T12:52:27+02:00 core: add case functions for str_hash get and set --- Modified: src/core/str_hash.h --- Diff: https://github.com/kamailio/kamailio/commit/6ffe43a50cbc33bb83b14f3aef3341a… Patch: https://github.com/kamailio/kamailio/commit/6ffe43a50cbc33bb83b14f3aef3341a… --- diff --git a/src/core/str_hash.h b/src/core/str_hash.h index 8755a6272c..263bb3cd3d 100644 --- a/src/core/str_hash.h +++ b/src/core/str_hash.h @@ -1,5 +1,5 @@ /* - * Copyright (C) 2006 iptelorg GmbH + * Copyright (C) 2006 iptelorg GmbH * * Permission to use, copy, modify, and distribute this software for any * purpose with or without fee is hereby granted, provided that the above @@ -70,17 +70,17 @@ inline static int str_hash_alloc(struct str_hash_table* ht, int size) inline static void str_hash_init(struct str_hash_table* ht) { int r; - + for (r=0; r<ht->size; r++) clist_init(&(ht->table[r]), next, prev); } -inline static void str_hash_add(struct str_hash_table* ht, +inline static void str_hash_add(struct str_hash_table* ht, struct str_hash_entry* e) { int h; - + h=get_hash1_raw(e->key.s, e->key.len) % ht->size; clist_insert(&ht->table[h], e, next, prev); } @@ -92,7 +92,7 @@ inline static struct str_hash_entry* str_hash_get(struct str_hash_table* ht, { int h; struct str_hash_entry* e; - + h=get_hash1_raw(key, len) % ht->size; clist_foreach(&ht->table[h], e, next){ if ((e->key.len==len) && (memcmp(e->key.s, key, len)==0)) @@ -101,6 +101,30 @@ inline static struct str_hash_entry* str_hash_get(struct str_hash_table* ht, return 0; } +inline static void str_hash_case_add(struct str_hash_table* ht, + struct str_hash_entry* e) +{ + int h; + + h=get_hash1_case_raw(e->key.s, e->key.len) % ht->size; + clist_insert(&ht->table[h], e, next, prev); +} + + + +inline static struct str_hash_entry* str_hash_case_get(struct str_hash_table* ht, + const char* key, int len) +{ + int h; + struct str_hash_entry* e; + + h=get_hash1_case_raw(key, len) % ht->size; + clist_foreach(&ht->table[h], e, next){ + if ((e->key.len==len) && (strncasecmp(e->key.s, key, len)==0)) + return e; + } + return 0; +} #define str_hash_del(e) clist_rm(e, next, prev)

1 0

RTPEngine - Support for Unified Plan SDP Format
by M S 19 May '20

19 May '20

Hi, Since Chrome M72, the default format of SDP generated by Chrome's WebRTC stack has been changed to "unified-plan" instead of Google's old "plan-b" format, which has no been deprecated and planned to be removed sometime in mid 2020. Here are more details about it. https://webrtc.org/getting-started/unified-plan-transition-guide This poses a big problem for non-web WebRTC clients, servers and proxies (such as RTPEngine) which will stop working as soon as Google Chrome removes support for legacy "plan-b" SDP format. In kmaialio's RTPEngine module, so far I have seen no support for this new "unified-plan" SDP format. Are there any support planed for future? Meanwhile, if there any way to add support for it in Kamailio via SDP manipulation e.g. SDPOPS module etc. Thank you. -- Muhammad Shahzad Shafi Tel: +49 176 24 73 85 20

3 3

← Newer
1
...
15
16
17
18
19
20
21
...
40
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev May 2020