the patch was been back ported
https://github.com/kamailio/kamailio/commit/62e6271dba40a8bb0dc96992e37877bb9aabf9f8
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Module: kamailio
Branch: 5.0
Commit: 62e6271dba40a8bb0dc96992e37877bb9aabf9f8
URL:
https://github.com/kamailio/kamailio/commit/62e6271dba40a8bb0dc96992e37877bb9aabf9f8
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
Date: 2
when we set `PROC_NOCHLDINIT` on the consumer workers, the module cannot bind
to the presentity table. Perhaps @miconda would know why this is the case, but
this is the error:
```
ERROR: [db.c:452]: db_use_table(): invalid parameter value [presentity]
ERROR: nsq [nsq_pua.c:146]:
Closed #905.
--
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/905#event-918055574___
sr-dev mailing list
sr-dev@lists.sip-router.org
Merged #921.
--
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/921#event-918054419___
sr-dev mailing list
sr-dev@lists.sip-router.org
Module: kamailio
Branch: master
Commit: 6f0c96da5a5caa8197132f6a2d5ffb3fc2123340
URL:
https://github.com/kamailio/kamailio/commit/6f0c96da5a5caa8197132f6a2d5ffb3fc2123340
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
@oej
I've submitted a PR for this issue
https://github.com/kamailio/kamailio/pull/921
please review. If i missed anything, can you please point out where?
Thanks,
Emmanuel
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/921
-- Commit Summary --
* nsq: update doc
-- File Changes --
M src/modules/nsq/doc/nsq_admin.xml (176)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/921.patch
Merged #884.
--
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/884#event-887465885___
sr-dev mailing list
sr-dev@lists.sip-router.org
Module: kamailio
Branch: master
Commit: 6d0ec36a4a3b721de2a05680a2db230600e4e494
URL:
https://github.com/kamailio/kamailio/commit/6d0ec36a4a3b721de2a05680a2db230600e4e494
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/884
-- Commit Summary --
* nsq: use max_in_flight value
-- File Changes --
M src/modules/nsq/nsq_mod.c (12)
-- Patch Links --
Merged #866.
--
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/866#event-875324353___
sr-dev mailing list
sr-dev@lists.sip-router.org
Module: kamailio
Branch: master
Commit: 17fadef3127901a08daf115fb350c2e59b1fd485
URL:
https://github.com/kamailio/kamailio/commit/17fadef3127901a08daf115fb350c2e59b1fd485
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/866
-- Commit Summary --
* nsq: add db_table_lock_type
-- File Changes --
M modules/nsq/doc/nsq_admin.xml (19)
M modules/nsq/nsq_mod.c (5)
M modules/nsq/nsq_mod.h (2)
Module: kamailio
Branch: master
Commit: ea2ce2da94927a5a555ef52c25f7426e59c361b7
URL:
https://github.com/kamailio/kamailio/commit/ea2ce2da94927a5a555ef52c25f7426e59c361b7
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 24d71219e44f24c81c3058bf7e8c1f4ea2996f85
URL:
https://github.com/kamailio/kamailio/commit/24d71219e44f24c81c3058bf7e8c1f4ea2996f85
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/854
-- Commit Summary --
* presence: cleanup whitespace in notify.c
* presence: add parameter fix_startup
-- File Changes --
M modules/presence/doc/presence_admin.xml (22)
it's not entirely clear to me what this PR fixes. What about adding a module
parameter (default = enabled) to enable this code?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
@lazedo
you can see that we sending expires=1 in our NSQ message and in nsq_pua.c
```C
struct json_object *ExpiresObj = nsq_json_get_object(json_obj,
BLF_JSON_EXPIRES);
if (ExpiresObj != NULL) {
expires = json_object_get_int(ExpiresObj);
if
@miconda yes, we are seeing this issue during runtime. not on startup
we reverted the commit and problem went away
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
this commit broke presence for us. we use NSQ + presence, after a phone reboots
and we send a NOTIFY with confirmed state, presence module sends a NOTIFY with
terminated state.
please revert this commit. see the log and SIP packet below:
```
Nov 3 09:23:52 /usr/local/sbin/kamailio[26288]:
Module: kamailio
Branch: master
Commit: c73a7eab210c7806c9d5fadacac1d3acc0ad6580
URL:
https://github.com/kamailio/kamailio/commit/c73a7eab210c7806c9d5fadacac1d3acc0ad6580
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: cf8d6066afc102d5904dddec0275079885f0ead1
URL:
https://github.com/kamailio/kamailio/commit/cf8d6066afc102d5904dddec0275079885f0ead1
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/843
-- Commit Summary --
* janssonrpc-c: fix shared memory leak on notification requests
-- File Changes --
M modules/janssonrpc-c/janssonrpc_io.c (2)
-- Patch Links --
Merged #829.
--
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/829#event-834592550___
sr-dev mailing list
sr-dev@lists.sip-router.org
Module: kamailio
Branch: master
Commit: c6b38e437387aba003bef8ef9496d5a7568e9c97
URL:
https://github.com/kamailio/kamailio/commit/c6b38e437387aba003bef8ef9496d5a7568e9c97
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
@miconda removed contributor remark
--
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/829#issuecomment-255847230___
sr-dev mailing list
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/829
-- Commit Summary --
* janssonrpc-c: fix memory leak when using janssonrpc_notification
-- File Changes --
M modules/janssonrpc-c/janssonrpc_io.c (19)
-- Patch Links --
Merged #745.
--
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/745#event-754453896___
sr-dev mailing list
sr-dev@lists.sip-router.org
-remove local libnsq files
-pass NULL for NSQReaderCfg to new_nsq_reader()
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/745
-- Commit Summary --
* remove reader.c from module and use libnsq reader.c
* remove nsq.h in favor of
@miconda @lazedo
I'm canceling this PR for now.
We are testing some settings on Polycom and we want to further analyze before
we are sure this code is needed, thanks.
Emmanuel
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on
Closed #654.
---
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/654#event-686451361___
sr-dev mailing list
sr-dev@lists.sip-router.org
@lazedo
update: our Polycom on staging is losing presence with the following
configuration
```
modparam("presence", "expires_offset", 120)
//modparam("presence", "subscribe_expires_offset", 60)
```
---
You are receiving this because you are subscribed to this thread.
Reply to this email
@lazedo
I removed expires_offset so now i have:
```
#modparam("presence", "expires_offset", 60)
modparam("presence", "subscribe_expires_offset", 60)
```
I will test the following on our staging
```
modparam("presence", "expires_offset", 120)
#modparam("presence", "subscribe_expires_offset", 60)
@lazedo
According to the doc on `expires_offset`: The value in seconds that should be
subtracted from the expires value when sending a 200OK for a publish. It is
used for forcing the client to send an update before the old publish expires.
This is not what we are attempting to resolve with
@lazedo It does not. Reading the doc, `expires_offset` seems to only affect
PUBLISH requests.
Testing and examining the code, I found that `expires_offset` had not affect on
SUBSCRIBE requests.
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or
@miconda
This resolved an on-going issue we noticed with BLF line keys losing presence
(on polycoms we saw the line keys turning gray). We noticed it was happening
because the Polycom was not re-subscribing before the expiration and therefore,
kamailio was deleting the subscription from the
* defaults to 0
* adds X seconds to expires before being stored in htable or database
* causes subscribes to not be removed prematurely
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/654
-- Commit Summary --
* presence: add new
Module: kamailio
Branch: master
Commit: c31bb61739c9d38c7a489a5562861e2ee5dcf503
URL:
https://github.com/kamailio/kamailio/commit/c31bb61739c9d38c7a489a5562861e2ee5dcf503
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 64bab0c38596c9dea13a22a5d2a89c5e24238fe2
URL:
https://github.com/kamailio/kamailio/commit/64bab0c38596c9dea13a22a5d2a89c5e24238fe2
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: e7eb7a24d0ceaf77803ff2d9c798a9b3247b89ff
URL:
https://github.com/kamailio/kamailio/commit/e7eb7a24d0ceaf77803ff2d9c798a9b3247b89ff
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: c5b79b55f2f0cdaf7f99e5f0d8042ba6435b98c8
URL:
https://github.com/kamailio/kamailio/commit/c5b79b55f2f0cdaf7f99e5f0d8042ba6435b98c8
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 1aaac266dcafcb083311f3d9d871897c8130203a
URL:
https://github.com/kamailio/kamailio/commit/1aaac266dcafcb083311f3d9d871897c8130203a
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Merged #613.
---
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/613#event-657308238___
sr-dev mailing list
sr-dev@lists.sip-router.org
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/613
-- Commit Summary --
* nsq: fix segfault on module destroy
-- File Changes --
M modules/nsq/nsq_mod.c (8)
-- Patch Links --
Module: kamailio
Branch: master
Commit: 688728fc8b70697afb9ae79503eefc6c6da34481
URL:
https://github.com/kamailio/kamailio/commit/688728fc8b70697afb9ae79503eefc6c6da34481
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 2584ba37b2d8c0aa42f83820dd6384c3ed9c3042
URL:
https://github.com/kamailio/kamailio/commit/2584ba37b2d8c0aa42f83820dd6384c3ed9c3042
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 0770773625c2197a0f94b7b31b593b714fc2d259
URL:
https://github.com/kamailio/kamailio/commit/0770773625c2197a0f94b7b31b593b714fc2d259
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 45a4848b0275be9aded61d2c0f3cc9908a2b2575
URL:
https://github.com/kamailio/kamailio/commit/45a4848b0275be9aded61d2c0f3cc9908a2b2575
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Merged #608.
---
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/608#event-655061119___
sr-dev mailing list
sr-dev@lists.sip-router.org
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/608
-- Commit Summary --
* nsq: update comment to include recognition for 2600hz
-- File Changes --
M modules/nsq/nsq_mod.c (3)
-- Patch Links --
Merged #607.
---
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/607#event-654946613___
sr-dev mailing list
sr-dev@lists.sip-router.org
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/607
-- Commit Summary --
* nsq: allow sending MWI-Voice-Message json for message details
-- File Changes --
M modules/nsq/defs.h (2)
M modules/nsq/nsq_pua.c (16)
-- Patch
Module: kamailio
Branch: master
Commit: 80f1eeb2a09c5900e392383d7c7a13641d0c9928
URL:
https://github.com/kamailio/kamailio/commit/80f1eeb2a09c5900e392383d7c7a13641d0c9928
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 18029a11e8eddcde059a8342669125a488ee5d7a
URL:
https://github.com/kamailio/kamailio/commit/18029a11e8eddcde059a8342669125a488ee5d7a
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: d76f024bca961b490d49874f876cfa763fcf0803
URL:
https://github.com/kamailio/kamailio/commit/d76f024bca961b490d49874f876cfa763fcf0803
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: d332b284aa38c31d18338bddbdeeb51da3813353
URL:
https://github.com/kamailio/kamailio/commit/d332b284aa38c31d18338bddbdeeb51da3813353
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: ecb28fd22203600181ff6f3d5c21625cc8d11a37
URL:
https://github.com/kamailio/kamailio/commit/ecb28fd22203600181ff6f3d5c21625cc8d11a37
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Merged #603.
---
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/603#event-653434253___
sr-dev mailing list
sr-dev@lists.sip-router.org
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/603
-- Commit Summary --
* presence: fix typo in presence_admin.xml
-- File Changes --
M modules/presence/doc/presence_admin.xml (2)
-- Patch Links --
- deletes expired presentity records without sending a NOTIFY
- `force_delete` disabled by default
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/601
-- Commit Summary --
* presence: introduce new module parameter `force_delete`
@miconda here is PR for new presence parameter
---
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/601#issuecomment-217447374___
sr-dev
@miconda
I'd like to introduce a new parameter to the presence module, `force_delete`
This parameter (disabled by default) will delete (not mark for deletion)
expired presentity records without sending a NOTIFY.
We control presence using the NSQ module and found the presence module was
After further investigation of the presence module, I found that the function
inside publish.c will delete (instead of mark for deletion) records if there
are no watchers.
For our purposes, we will not be using watchers so I'd like to propose a
presence module setting that will force deletion
We are using the `presence` module and have noticed many errors in the log as
follows:
```WARNING: db_postgres [km_dbase.c:240]: db_postgres_submit_query(): postgres
result check failed with code 7 (PGRES_FATAL_ERROR)
WARNING: db_postgres [km_dbase.c:244]: db_postgres_submit_query(): postgres
Module: kamailio
Branch: master
Commit: 0f2fddb3f5ec139d0e87318a784dfb4258b0e25e
URL:
https://github.com/kamailio/kamailio/commit/0f2fddb3f5ec139d0e87318a784dfb4258b0e25e
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 9e5587351d6e4739d44e47fd426e5d091822
URL:
https://github.com/kamailio/kamailio/commit/9e5587351d6e4739d44e47fd426e5d091822
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 3dee74bbb7eca8339108ce3dc89c1f76b4d560cc
URL:
https://github.com/kamailio/kamailio/commit/3dee74bbb7eca8339108ce3dc89c1f76b4d560cc
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 67126a1f7ad714d316f13fce73b68342c820e0a2
URL:
https://github.com/kamailio/kamailio/commit/67126a1f7ad714d316f13fce73b68342c820e0a2
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Module: kamailio
Branch: master
Commit: 8ef73bd12ac4ef8cb3b2881438bff8e525237dbf
URL:
https://github.com/kamailio/kamailio/commit/8ef73bd12ac4ef8cb3b2881438bff8e525237dbf
Author: Emmanuel Schmidbauer <emman...@getweave.com>
Committer: Emmanuel Schmidbauer <emman...@getweave.com>
D
Merged #590.
---
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/590#event-644347953___
sr-dev mailing list
sr-dev@lists.sip-router.org
- corrected transformations section, nsq.json usage
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/590
-- Commit Summary --
* nsq: updated doc
-- File Changes --
M modules/nsq/doc/nsq_admin.xml (8)
-- Patch Links --
Merged #582.
---
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/582#event-641468731___
sr-dev mailing list
sr-dev@lists.sip-router.org
@miconda
This is the module I mentioned to receive messages from NSQ (http://nsq.io/)
It's similar to the kazoo module.
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/582
-- Commit Summary --
* New Module NSQ
-- File Changes --
A modules/nsq/Makefile (23)
A modules/nsq/README.md (312)
A modules/nsq/defs.h (106)
A
Module: kamailio
Branch: master
Commit: 93fd86c882fd82f4277a80322dd729c2dc0370a0
URL:
https://github.com/kamailio/kamailio/commit/93fd86c882fd82f4277a80322dd729c2dc0370a0
Author: Emmanuel Schmidbauer <eschmidba...@voipxswitch.com>
Committer: Emmanuel Schmidbauer <eschmidba...@voipxs
Please see PR
https://github.com/kamailio/kamailio/pull/546
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/533#issuecomment-196375832___
sr-dev mailing list
sr-dev@lists.sip-router.org
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/546
-- Commit Summary --
* Fix memory leak by freeing RURI in trusted table struct; issue #533
-- File Changes --
M modules/permissions/hash.c (1)
-- Patch Links --
Hello,
I found the memory leak in permissions.
Can you please push this patch?
```
diff --git a/modules/permissions/hash.c b/modules/permissions/hash.c
index cfc81fe..84b20e2 100644
--- a/modules/permissions/hash.c
+++ b/modules/permissions/hash.c
@@ -423,6 +423,7 @@ void empty_hash_table(struct
Hello,
We've noticed a memory leak caused by reloading the trusted table data.
kamctl trusted reload
We verified this by running script to reload the data frequently and captured
memory usage using kex + corex.
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
Hello,
When we have 64 or more records in same 'grp'
kamcmd permissions.addressDump fails with error: 500 - Internal error creating
rpc ih
I'm guessing this is a problem with the size of the *ih variable. Can it be
increased to accommodate larger datasets?
Thanks,
Emmanuel
---
Reply to this
Hello,
I found that his was a problem with db_mode set to 0
I changed db_mode to 1 and everything works.
I would still consider this a bug as it causes segfault. Perhaps an error
message should be output instead of crashing.
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
I don't think this is related to recent changes to permissions module. I tested
on version 4.3.0 before recent changes and same result
Tested on version: kamailio 4.3.0 (x86_64/linux) 5e9862
---
Reply to this email directly or view it on GitHub:
When we reload trusted table and using MySQL as database, kamailio crashes.
This does not happen when using PostgreSQL
We only have one record in the database too:
mysql select * from trusted\G
*** 1. row ***
id: 1
src_ip:
here is debugging log
# kamcmd permissions.trustedReload
8(5433) DEBUG: ctl [../../io_wait.h:376]: io_watch_add(): DBG:
io_watch_add(0x7fa9ddf08720, 9, 3, 0x2890680), fd_no=1
8(5433) DEBUG: ctl [io_listener.c:442]: handle_new_connect(): handle_stream
read: new connection (1) on
(gdb) bt full
#0 0x7f17e53d3832 in reload_trusted_table () at trusted.c:96
cols = {0x7f17e55e7800, 0x7f17e55e7810, 0x7f17e55e7820, 0x7f17e55e7830,
0x7f17e55e7840, 0x7f17e55e7850}
res = 0x7f17e8d9e530
row = 0x7f17e9898ea8
val = 0x1010
new_hash_table =
Module: kamailio
Branch: master
Commit: 90e0bec635abaed2bed38bd9b98e16d9895618f4
URL:
https://github.com/kamailio/kamailio/commit/90e0bec635abaed2bed38bd9b98e16d9895618f4
Author: Emmanuel Schmidbauer eschmidba...@voipxswitch.com
Committer: Emmanuel Schmidbauer eschmidba...@voipxswitch.com
Date
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/217
-- Commit Summary --
* added doc for proirites inside permission's module
-- File Changes --
M modules/permissions/doc/permissions_admin.xml (6)
-- Patch Links --
Please see new commit message.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/217#issuecomment-113515364___
sr-dev mailing list
sr-dev@lists.sip-router.org
Hello,
made changes as you requested.
Please see latest commit.
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112444793___
sr-dev mailing list
Hello,
You can download the full git diff (it applies clean to master branch) here
http://paste.debian.net/download/231826
Thank you.
Emmanuel
---
Reply to this email directly or view it on GitHub:
Thank you for the response.
Yes, I did re-order the fields to make it more comprehensive in the long run.
I've been running this patch on my test machine for several weeks without any
issues.
I hope you will consider reviewing and testing soon as we want to put this in
production.
Thanks,
Hello,
Any chance this can be reviewed before the feature request window closes?
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-110356090___
sr-dev mailing list
Hello,
This pull request adds 2 features.
--Prioritize the trusted records (highest to lowest priority)
--Match from-uri AND request-uri, if either is blank only one is matched.
Please review the pull request and let me know if any other changes are needed.
Thank you.
Emmanuel
You can view,
Patch fixed it!
Thanks for the quick response.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/137#issuecomment-95607717___
sr-dev mailing list
sr-dev@lists.sip-router.org
Closed #137.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/137#event-288375612___
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
I'm running latest kamailio version (as of today) from the git
Using postgres 9.4 db backend.
When I do trustedReload output is as follows:
root@ua01-proxy01:~# kamctl kamcmd permissions.trustedReload
error: 500 - Reload failed.
and kamailio log shows:
Apr 22 10:16:21 ua01-proxy01
Thanks, that fixed it.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/138#issuecomment-95610578___
sr-dev mailing list
sr-dev@lists.sip-router.org
Closed #138.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/138#event-288388281___
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
I'm running latest kamailio version (as of today) from the git
Using postgres 9.4 db backend.
When I do trustedDump output is strange.
Example:
root@ua01-proxy01:~# kamctl kamcmd permissions.trustedDump
{
table: 27
item: {
ip: 10.10.5.112úæ¼
100 matches
Mail list logo
