[sr-dev] Re: [kamailio/kamailio] multiple subsytems: unsigned integer conversion fixes (PR #3828)

2024-06-11 Thread Tyler Moore via sr-dev
@miconda just wanted to post back real quick on this one, I think this might take a little while to review... ``` ~/projects/kamailio|[git:sr_3.1_freeze:master !]$ grep -RE -m 1 -l 'db_val2pv_spec|db1_res_t' src/modules/ | cut -d '/' -f 3 | sort -u alias_db auth_db avpops carrierroute cplc db_be

[sr-dev] PR 3828 Request For Comments

2024-06-04 Thread tyler moore via sr-dev
Hey Team, I submitted a PR a while back that has not had much discussion: https://github.com/kamailio/kamailio/pull/3828 The intention was to allow loading in unsigned integer IDs from DB into PVs. Currently converting unsigned integers from DB to PVs will error by default unless the module spe

[sr-dev] Re: RPM Build Toolchain

2024-05-29 Thread tyler moore via sr-dev
t; and only found the build commands for the source RPMs. On 5/29/24 08:03, Sergey Safarov wrote: Hi Tyler. I have rebuilt RPM packages. Please clean repo cache and reinstall packages. dnf reinstall kamailio* On Tue, May 28, 2024 at 6:39 PM tyler moore via sr-dev wrote: Sergey,

[sr-dev] Re: RPM Build Toolchain

2024-05-28 Thread tyler moore via sr-dev
Sergey, Any chance we could generate a new build of the RPM including the updated version of openssl? Regards, Tyler Moore Full Stack Software Engineer dOpenSource Office: 888-907-2085, ext: 34 Cell: 248-909-2769 Email:tmo...@dopensource.com ___ Kam

[sr-dev] RPM Build Toolchain

2024-05-21 Thread tyler moore via sr-dev
Sergey, Is the build toolchain for the RPM packages using an old version of openssl? I am seeing incompatibility errors when testing with 5.8.1 RPM and openssl-devel from system, or appstream repos. These are the version differences I see in the logs: |CRITICAL: tls [tls_init.c:870]: tls_h_mo

[sr-dev] Re: [kamailio/kamailio] multiple subsytems: unsigned integer conversion fixes (PR #3828)

2024-05-08 Thread Tyler Moore via sr-dev
Morning Team, Checking in on this PR. Can we get this assigned? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3828#issuecomment-2100723964 You are receiving this because you are subscribed to this thread. Message ID: _

[sr-dev] Re: [kamailio/kamailio] uac: add reload_delta parameter (PR #3829)

2024-05-08 Thread Tyler Moore via sr-dev
Morning Team, Checking in on this PR. Can we get this assigned? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3829#issuecomment-2100724472 You are receiving this because you are subscribed to this thread. Message ID: _

[sr-dev] Re: [kamailio/kamailio] multiple subsytems: unsigned integer conversion fixes (PR #3828)

2024-04-29 Thread Tyler Moore via sr-dev
> Thanks for the PR. Could you please fix the format by running clang-format on > the files and do a force-push to update the individual commits? This check is > right now failing. Fixed the formatting, let me know if further tests need provided for this PR. -- Reply to this email directly or

[sr-dev] Re: [kamailio/kamailio] uac: add reload_delta parameter (PR #3829)

2024-04-29 Thread Tyler Moore via sr-dev
> Thanks for the PR. Could you please fix the format by running clang-format on > the files and do a force-push to update the individual commit(s)? This check > is right now failing. Fixed formatting, let me know if any other changes are needed. -- Reply to this email directly or view it on Gi

[sr-dev] Re: [kamailio/kamailio] uac: add reload_delta parameter (PR #3829)

2024-04-26 Thread Tyler Moore via sr-dev
@devopsec pushed 1 commit. 9650c121c66263ef51c1414acb36994c49caee9c uac: add reload_delta parameter -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3829/files/fb61a4281c0a2a60d527f18c269c0e8eaac23476..9650c121c66263ef51c1414acb36994c49caee9c You are receiving this because you ar

[sr-dev] Re: [kamailio/kamailio] multiple subsytems: unsigned integer conversion fixes (PR #3828)

2024-04-26 Thread Tyler Moore via sr-dev
@devopsec pushed 1 commit. db042404783262c79271ac6fd2e0b9f1f5010822 lib/srdb1: add uint support for db->pv conversions -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3828/files/f19e574beebff9b1f17f8ab63a7cf5478974eb11..db042404783262c79271ac6fd2e0b9f1f5010822 You are receiving

[sr-dev] Re: [kamailio/kamailio] utils/kamctl: make root host/port configurable (PR #3830)

2024-04-25 Thread Tyler Moore via sr-dev
## background We found that mariadb ver >= 10.6.1 --port= now defaults to transport=tcp. Using mysql client to connect to a unix socket --port= should **not** be provided. To support this in kamdbctl the port / host is only added to the command if *set* in kamctlrc. ## tests Tested with ma

[sr-dev] [kamailio/kamailio] utils/kamctl: make root host/port configurable (PR #3830)

2024-04-25 Thread Tyler Moore via sr-dev
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 mo

[sr-dev] [kamailio/kamailio] uac: add reload_delta parameter (PR #3829)

2024-04-25 Thread Tyler Moore via sr-dev
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 mo

[sr-dev] Re: [kamailio/kamailio] Uint fixes (PR #3828)

2024-04-25 Thread Tyler Moore via sr-dev
For testing this was validated on master with the setup below. ## environment ``` root@kam:~# uname -a Linux kam 6.1.0-9-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.27-1 (2023-05-08) x86_64 GNU/Linux ``` ``` root@kam:~# lsb_release -a Distributor ID: Debian Description:Debian GNU/Linux 12 (book

[sr-dev] [kamailio/kamailio] Uint fixes (PR #3828)

2024-04-25 Thread Tyler Moore via sr-dev
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 mo

[sr-dev] Re: [kamailio/kamailio] TLS Module Regression (ee key too small) (Issue #3737)

2024-02-02 Thread Tyler Moore via sr-dev
added default provider and that seemed to fix the issue. pertinent sections from /etc/ssl/openssl.cnf: ``` [openssl_init] providers = provider_sect [provider_sect] default = default_sect ``` thanks for pointing out that related issue @space88man I think this version of openssl.cnf is packag

[sr-dev] Re: [kamailio/kamailio] TLS Module Regression (ee key too small) (Issue #3737)

2024-02-02 Thread Tyler Moore via sr-dev
Not sure if this helps but the cert/key was generated with: ``` penssl req -new -newkey rsa:4096 -x509 -sha256 -days 365 -nodes -out /etc/dsiprouter/certs/dsiprouter-cert.pem -keyout /etc/dsiprouter/certs/dsiprouter-key.pem -subj "/C=US/ST=MI/L=Detroit/O=dSIPRouter/CN=ec2-3-94-130-255.compute-1

[sr-dev] Re: [kamailio/kamailio] TLS Module Regression (ee key too small) (Issue #3737)

2024-02-02 Thread Tyler Moore via sr-dev
Same vanilla version of `ca-certificates`: ``` root@ip-172-31-22-12:~# dpkg -l |grep ca-cert ii ca-certificates 20230311 all Common CA certificates ``` and just to verify the same number of certs: ``` root@ip-172-31-22-12:~# ls -l /etc/ssl/certs/

[sr-dev] Re: [kamailio/kamailio] TLS Module Regression (ee key too small) (Issue #3737)

2024-02-01 Thread Tyler Moore via sr-dev
Sounds like the code path that is causing the error is not being run. Is that on Debian 12? Try with the following tls.cfg: ``` [server:default] method = TLSv1.2+ verify_certificate = yes require_certificate = yes private_key = /etc/ssl/private/dsiprouter-key.pem certificate = /etc/ssl/private

[sr-dev] Re: [kamailio/kamailio] TLS Module Regression (ee key too small) (Issue #3737)

2024-02-01 Thread Tyler Moore via sr-dev
@space88man I shared a link below with an example key/cert pair from an ephemeral VM. I can provide access to that VM for testing if needed as well. https://drive.google.com/drive/folders/1255E1iSUKaKbElwDsC8o3qsh6YTN5jrJ?usp=drive_link -- Reply to this email directly or view it on GitHub: ht

[sr-dev] [kamailio/kamailio] TLS Module Regression (ee key too small) (Issue #3737)

2024-01-31 Thread Tyler Moore via sr-dev
### Description The TLS module seems to have some regression from 5.7.3 to 5.7.4 causing `tls.reload` to fail loading certificates. ### Troubleshooting System Info - kamailio version: 5.7.4 (from official kamailio repos) - distro version: debian 12 - OS/kernel version: Linux ip-172-31-30-

[sr-dev] Re: [kamailio/kamailio] dispatcher: fix weighted sort for n<100 calls (PR #3531)

2023-10-05 Thread Tyler Moore via sr-dev
@miconda i'll open this as a new PR when I can get back around to it, thanks! -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3531#issuecomment-1748937640 You are receiving this because you are subscribed to this thread. Message ID: ___

[sr-dev] Re: [kamailio/kamailio] dispatcher: fix weighted sort for n<100 calls (PR #3531)

2023-10-05 Thread Tyler Moore via sr-dev
Closed #3531. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3531#event-10564114959 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development Mailing