[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2023-01-06 Thread Daniel-Constantin Mierla
OK, open a new issue if any problem is found. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3318#issuecomment-1373508514 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2023-01-06 Thread Daniel-Constantin Mierla
Closed #3318 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3318#event-8168519770 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2023-01-05 Thread Ben Kaufman
Thank you. I'll try to create a testing situation around this. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3318#issuecomment-1372373543 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2023-01-05 Thread Daniel-Constantin Mierla
Probably the developer thought only one of the tables are used in a Kamailio instance, they kind of overlap a lot in purpose and functionality. Personally I use only address table, I don't think I ever initiated a Kamailio deployment using the trusted table. Anyhow, because this kind of

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2022-12-28 Thread Alex Balashov
Ah, okay. Personally, it seems to me the real problem is in the documentation's claim that `reload_delta` creates separation between "RPC reloads", vs. no clear conception of what an "RPC reload" is. In thinking about it generally, I would expect such a reload to encompass both tables. --

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2022-12-28 Thread Henning Westerholt
The reload_delta functionality was added because it was observed that excessive reloads (e.g. every few seconds) could cause issues under load, as mentioned in the docs. A workaround for your cfg is to add two individual timers for both reload commands, but this might cause some small drift

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2022-12-28 Thread Ben Kaufman
I'm calling them separately because there's no function to call them together. ;) -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3318#issuecomment-1366809182 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] Permissions module `reload_delta` is not scoped to individual tables. (Issue #3318)

2022-12-28 Thread Alex Balashov
You would be correct in that `rpc_trusted_reload()` and `rpc_address_reload()` call the same `rpc_check_reload()` function, which uses a single timer. So, calling a reload on either of those would reset the timer. But why do you want to reload them separately? Does it matter? -- Reply to