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

2024-07-31 Thread Daniel-Constantin Mierla via sr-dev
Merged #3828 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3828#event-13720168308 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Developm

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

2024-07-23 Thread github-actions[bot] via sr-dev
This PR is stale because it has been open 6 weeks with no activity. Remove stale label or comment or this will be closed in 2 weeks. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3828#issuecomment-2246758587 You are receiving this because you are

[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] Re: [kamailio/kamailio] multiple subsytems: unsigned integer conversion fixes (PR #3828)

2024-06-05 Thread Daniel-Constantin Mierla via sr-dev
As I get it from the patches, it adds support to permissions and htable modules. The latests versions have LONG as type for PV value, isn't it cover the unsigned int range on 64b systems (which should be the vast majority these days)? After all, it could be merged, I was reticent when it was su

[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] 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] 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] multiple subsytems: unsigned integer conversion fixes (PR #3828)

2024-04-25 Thread Henning Westerholt 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. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3828#issuecomment-207868