sr-dev August 2024

sr-dev@lists.kamailio.org

10 participants
163 discussions

git:master:ed6730aa: pua_reginfo: free pkg memory after getting the content of each XML attribute
by Victor Seva 27 Aug '24

27 Aug '24

Module: kamailio Branch: master Commit: ed6730aa64e0e7c1580584d6f808e76a665e5882 URL: https://github.com/kamailio/kamailio/commit/ed6730aa64e0e7c1580584d6f808e76… Author: Andrii Pogrebennyk <andrii(a)ng-voice.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-08-27T09:35:47+02:00 pua_reginfo: free pkg memory after getting the content of each XML attribute - free the memory from the state XML attribute that was returned and immediately used in another function call without freeing leaving one small mem leak; - add comments but keep the dead code regarding parsed but unused attrs. --- Modified: src/modules/pua_reginfo/notify.c --- Diff: https://github.com/kamailio/kamailio/commit/ed6730aa64e0e7c1580584d6f808e76… Patch: https://github.com/kamailio/kamailio/commit/ed6730aa64e0e7c1580584d6f808e76…

1 0

[kamailio/kamailio] github: [skip ci]: bump tj-actions/changed-files from 44 to 45 (PR #3957)
by dependabot[bot] 27 Aug '24

27 Aug '24

Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 44 to 45. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/tj-actions/changed-files/releases">tj… releases</a>.</em></p> <blockquote> <h2>v45</h2> <h1>Changes in v45.0.0</h1> <h2>🔥🔥 BREAKING CHANGE 🔥🔥</h2> <ul> <li>With changes detected using GitHub's API setting <code>output_renamed_files_as_deleted_and_added</code> to <code>true</code>, would now include the previous file name in the list of deleted files instead of the current file name.</li> </ul> <h2>What's Changed</h2> <ul> <li>chore(deps): update dependency <code>@types/node</code> to v22.0.1 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2219&q… <li>Upgraded to v44.5.7 by <a href="https://github.com/tj-actions-bot"><code>@tj-acti… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2218&q… <li>chore(deps): update dependency ts-jest to v29.2.4 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2221&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.0.3 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2222&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.1.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2223&q… <li>chore(deps): update dependency eslint-plugin-jest to v28.7.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2224&q… <li>chore(deps): update dependency eslint-plugin-jest to v28.8.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2227&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.2.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2229&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.3.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2230&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.4.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2231&q… <li>fix(deps): update dependency <code>@octokit/rest</code> to v21.0.2 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2232&q… <li>chore(deps): lock file maintenance by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2226&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.4.1 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2233&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.4.2 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2237&q… <li>fix: remove unsupported input by <a href="https://github.com/jackton1"><code>@jackton1</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2238&q… <li>chore: conditionally skip certain test for foked prs by <a href="https://github.com/jackton1"><code>@jackton1</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2240&q… <li>chore: debug env vars by <a href="https://github.com/jackton1"><code>@jackton1</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2241&q… <li>fix: update test skip logic to use custom env by <a href="https://github.com/jackton1"><code>@jackton1</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2242&q… <li>Renamed Files decomposed into Deleted: previous_filename and Added:filename by <a href="https://github.com/Whadup"><code>@Whadup</code… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2236&q… <li>docs: add Whadup as a contributor for code by <a href="https://github.com/allcontributors"><code>@allcon… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2239&q… <li>Updated README.md by <a href="https://github.com/tj-actions-bot"><code>@tj-acti… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2243&q… </ul> <h2>New Contributors</h2> <ul> <li><a href="https://github.com/Whadup"><code>@Whadup</code… made their first contribution in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2236&q… </ul> <p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v44...v45.0.… <hr /> <h2>v45.0.0</h2> <h2>🔥🔥 BREAKING CHANGE 🔥🔥</h2> <ul> <li>With changes detected using GitHub's API setting <code>output_renamed_files_as_deleted_and_added</code> to <code>true</code>, would now include the previous file name in the list of deleted files.</li> </ul> <h2>What's Changed</h2> <ul> <li>chore(deps): update dependency <code>@types/node</code> to v22.0.1 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2219&q… <li>Upgraded to v44.5.7 by <a href="https://github.com/tj-actions-bot"><code>@tj-acti… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2218&q… <li>chore(deps): update dependency ts-jest to v29.2.4 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2221&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.0.3 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2222&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.1.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2223&q… <li>chore(deps): update dependency eslint-plugin-jest to v28.7.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2224&q… <li>chore(deps): update dependency eslint-plugin-jest to v28.8.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2227&q… <li>chore(deps): update dependency <code>@types/node</code> to v22.2.0 by <a href="https://github.com/renovate"><code>@renovate</… in <a href="https://redirect.github.com/tj-actions/changed-files/pull/2229&q… </ul>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/tj-actions/changed-files/blob/main/HISTORY.md… changelog</a>.</em></p> <blockquote> <h1>Changelog</h1> <h1><a href="https://github.com/tj-actions/changed-files/compare/v44.5.7...v4… - (2024-08-21)</h1> <h2>🐛 Bug Fixes</h2> <ul> <li>Update test skip logic to use custom env (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2242…) (<a href="https://github.com/tj-actions/changed-files/commit/b4c09741612c0…) - (Tonye Jack)</li> <li>Remove unsupported input (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2238…) (<a href="https://github.com/tj-actions/changed-files/commit/525aabbe28ff9…) - (Tonye Jack)</li> <li><strong>deps:</strong> Update dependency <code>@octokit/rest</code> to v21.0.2 (<a href="https://github.com/tj-actions/changed-files/commit/3ea85a1c0dd32…) - (renovate[bot])</li> </ul> <h2>➕ Add</h2> <ul> <li>Added missing changes and modified dist assets. (<a href="https://github.com/tj-actions/changed-files/commit/598b7212f2ea6…) - (GitHub Action)</li> </ul> <h2>📝 Rename</h2> <ul> <li>Renamed Files decomposed into Deleted: previous_filename and Added:filename (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2236… </ul> <p>Co-authored-by: Tonye Jack <a href="mailto:jtonye@ymail.com">jtonye@ymail.com</a> (<a href="https://github.com/tj-actions/changed-files/commit/f8034fbb842b9…) - (Lukas Pfahler)</p> <h2>🔄 Update</h2> <ul> <li>Updated README.md (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2243… </ul> <p>Co-authored-by: jackton1 <a href="mailto:17484350+jackton1@users.noreply.github.com">17484350+jackton1@users.noreply.github.com</a> Co-authored-by: GitHub Action <a href="mailto:action@github.com">action@github.com</a> (<a href="https://github.com/tj-actions/changed-files/commit/40853de9f8ce2…) - (tj-actions[bot])</p> <ul> <li>Update tsconfig.json (<a href="https://github.com/tj-actions/changed-files/commit/88201be733611…) - (Tonye Jack)</li> </ul> <h2>📚 Documentation</h2> <ul> <li>Add Whadup as a contributor for code (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2239…) (<a href="https://github.com/tj-actions/changed-files/commit/94b94fea13561…) - (allcontributors[bot])</li> </ul> <h2>⚙️ Miscellaneous Tasks</h2> <ul> <li>Debug env vars (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2241…) (<a href="https://github.com/tj-actions/changed-files/commit/246859db916b5…) - (Tonye Jack)</li> <li>Conditionally skip certain test for foked prs (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2240…) (<a href="https://github.com/tj-actions/changed-files/commit/46a4c027a6ca4…) - (Tonye Jack)</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.4.2 (<a href="https://github.com/tj-actions/changed-files/commit/25b256207da93…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.4.1 (<a href="https://github.com/tj-actions/changed-files/commit/257f12c17b624…) - (renovate[bot])</li> <li><strong>deps:</strong> Lock file maintenance (<a href="https://github.com/tj-actions/changed-files/commit/9b7ba7b107e24…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.4.0 (<a href="https://github.com/tj-actions/changed-files/commit/50693d8df5efe…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.3.0 (<a href="https://github.com/tj-actions/changed-files/commit/87d8f9488bd23…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.2.0 (<a href="https://github.com/tj-actions/changed-files/commit/950331d47c80d…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency eslint-plugin-jest to v28.8.0 (<a href="https://github.com/tj-actions/changed-files/commit/ed00b69562f42…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency eslint-plugin-jest to v28.7.0 (<a href="https://github.com/tj-actions/changed-files/commit/9d768c3d4e4de…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.1.0 (<a href="https://github.com/tj-actions/changed-files/commit/6b79c79980934…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.0.3 (<a href="https://github.com/tj-actions/changed-files/commit/99b60a2a91bd1…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency ts-jest to v29.2.4 (<a href="https://github.com/tj-actions/changed-files/commit/8ae850f052514…) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.0.1 (<a href="https://github.com/tj-actions/changed-files/commit/03d95b07db4ba…) - (renovate[bot])</li> </ul>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/tj-actions/changed-files/commit/40853de9f8ce2… Updated README.md (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2243… <li><a href="https://github.com/tj-actions/changed-files/commit/94b94fea13561… docs: add Whadup as a contributor for code (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2239… <li><a href="https://github.com/tj-actions/changed-files/commit/f8034fbb842b9… Renamed Files decomposed into Deleted: previous_filename and Added:filename (...</li> <li><a href="https://github.com/tj-actions/changed-files/commit/b4c09741612c0… fix: update test skip logic to use custom env (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2242… <li><a href="https://github.com/tj-actions/changed-files/commit/246859db916b5… chore: debug env vars (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2241… <li><a href="https://github.com/tj-actions/changed-files/commit/46a4c027a6ca4… chore: conditionally skip certain test for foked prs (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2240… <li><a href="https://github.com/tj-actions/changed-files/commit/525aabbe28ff9… fix: remove unsupported input (<a href="https://redirect.github.com/tj-actions/changed-files/issues/2238… <li><a href="https://github.com/tj-actions/changed-files/commit/25b256207da93… chore(deps): update dependency <code>@types/node</code> to v22.4.2</li> <li><a href="https://github.com/tj-actions/changed-files/commit/257f12c17b624… chore(deps): update dependency <code>@types/node</code> to v22.4.1</li> <li><a href="https://github.com/tj-actions/changed-files/commit/598b7212f2ea6… Added missing changes and modified dist assets.</li> <li>Additional commits viewable in <a href="https://github.com/tj-actions/changed-files/compare/v44...v45&qu… view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?d… Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3957 -- Commit Summary -- * github: [skip ci]: bump tj-actions/changed-files from 44 to 45 -- File Changes -- M .github/workflows/devcontainer.yaml (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3957.patch https://github.com/kamailio/kamailio/pull/3957.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3957 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3957(a)github.com>

2 1

git:master:1c5983c8: github: [skip ci]: bump tj-actions/changed-files from 44 to 45
by Victor Seva 27 Aug '24

27 Aug '24

Module: kamailio Branch: master Commit: 1c5983c8f7ab7a93b5c96efeaa9600b6f2246d13 URL: https://github.com/kamailio/kamailio/commit/1c5983c8f7ab7a93b5c96efeaa9600b… Author: dependabot[bot] <49699333+dependabot[bot](a)users.noreply.github.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-08-27T09:14:23+02:00 github: [skip ci]: bump tj-actions/changed-files from 44 to 45 Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 44 to 45. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](https://github.com/tj-actions/changed-files/compare/v44...v45) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support(a)github.com> --- Modified: .github/workflows/devcontainer.yaml --- Diff: https://github.com/kamailio/kamailio/commit/1c5983c8f7ab7a93b5c96efeaa9600b… Patch: https://github.com/kamailio/kamailio/commit/1c5983c8f7ab7a93b5c96efeaa9600b… --- diff --git a/.github/workflows/devcontainer.yaml b/.github/workflows/devcontainer.yaml index e70f646891c..3acd963b2df 100644 --- a/.github/workflows/devcontainer.yaml +++ b/.github/workflows/devcontainer.yaml @@ -33,14 +33,14 @@ jobs: - name: Get changed files if: ${{ ! inputs.force }} id: changed-files - uses: tj-actions/changed-files@v44 + uses: tj-actions/changed-files@v45 with: since_last_remote_commit: true - name: changes in relevant files if: ${{ ! inputs.force }} id: changed-files-builder - uses: tj-actions/changed-files@v44 + uses: tj-actions/changed-files@v45 with: files: | .devcontainer/**

1 0

git:master:9743c122: corex: set internal flag FL_VIA_NORECEIVED for forward_uac() variants
by Daniel-Constantin Mierla 26 Aug '24

26 Aug '24

Module: kamailio Branch: master Commit: 9743c1226e95e9a9f890d6c22ad1da7b9f9cc543 URL: https://github.com/kamailio/kamailio/commit/9743c1226e95e9a9f890d6c22ad1da7… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-26T18:07:47+02:00 corex: set internal flag FL_VIA_NORECEIVED for forward_uac() variants --- Modified: src/modules/corex/corex_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/9743c1226e95e9a9f890d6c22ad1da7… Patch: https://github.com/kamailio/kamailio/commit/9743c1226e95e9a9f890d6c22ad1da7… --- diff --git a/src/modules/corex/corex_mod.c b/src/modules/corex/corex_mod.c index cf663158fa0..7e121f8bb61 100644 --- a/src/modules/corex/corex_mod.c +++ b/src/modules/corex/corex_mod.c @@ -325,6 +325,7 @@ static int ki_forward_uac_uri(sip_msg_t *msg, str *vuri) return -1; } dst.proto = u->proto; + msg->msg_flags |= FL_VIA_NORECEIVED; ret = forward_request_mode( msg, &u->host, u->port_no, &dst, BUILD_NO_VIA1_UPDATE); if(ret >= 0) {

1 0

git:master:3acf1296: modules: readme files regenerated - siputils ... [skip ci]
by Kamailio Dev 26 Aug '24

26 Aug '24

Module: kamailio Branch: master Commit: 3acf129660be4e2d9f4dfa68ad3ec6d25c43fbf1 URL: https://github.com/kamailio/kamailio/commit/3acf129660be4e2d9f4dfa68ad3ec6d… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-08-26T15:46:11+02:00 modules: readme files regenerated - siputils ... [skip ci] --- Modified: src/modules/siputils/README --- Diff: https://github.com/kamailio/kamailio/commit/3acf129660be4e2d9f4dfa68ad3ec6d… Patch: https://github.com/kamailio/kamailio/commit/3acf129660be4e2d9f4dfa68ad3ec6d… --- diff --git a/src/modules/siputils/README b/src/modules/siputils/README index d3eda806d4d..f66c7391747 100644 --- a/src/modules/siputils/README +++ b/src/modules/siputils/README @@ -107,6 +107,7 @@ Gabriel Vasile 4.41. contact_param_rm(pname) 4.42. contact_param_check(pname) 4.43. hdr_date_check(tdiff) + 4.44. sip_parse_headers() 5. Exported pseudo-variables @@ -170,6 +171,7 @@ Gabriel Vasile 1.50. contact_param_rm usage 1.51. contact_param_check usage 1.52. hdr_date_check usage + 1.53. sip_parse_headers usage Chapter 1. Admin Guide @@ -238,6 +240,7 @@ Chapter 1. Admin Guide 4.41. contact_param_rm(pname) 4.42. contact_param_check(pname) 4.43. hdr_date_check(tdiff) + 4.44. sip_parse_headers() 5. Exported pseudo-variables @@ -455,6 +458,7 @@ modparam("siputils", "e164_max_len", 20) 4.41. contact_param_rm(pname) 4.42. contact_param_check(pname) 4.43. hdr_date_check(tdiff) + 4.44. sip_parse_headers() 4.1. options_reply() @@ -1203,6 +1207,19 @@ if (!hdr_date_check("10")) { } ... +4.44. sip_parse_headers() + + Return true if parsing (fast mode) all SIP headers is successful. + + This function can be used from ANY_ROUTE. + + Example 1.53. sip_parse_headers usage +... +if (sip_parse_headers()) { + ... +} +... + 5. Exported pseudo-variables 5.1. $pcv(all)

1 0

git:master:52036e27: siputils: docs for sip_parse_headers() function
by Daniel-Constantin Mierla 26 Aug '24

26 Aug '24

Module: kamailio Branch: master Commit: 52036e2772821459daea6d5ac98a3710d9daec13 URL: https://github.com/kamailio/kamailio/commit/52036e2772821459daea6d5ac98a371… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-26T15:39:20+02:00 siputils: docs for sip_parse_headers() function --- Modified: src/modules/siputils/doc/siputils_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/52036e2772821459daea6d5ac98a371… Patch: https://github.com/kamailio/kamailio/commit/52036e2772821459daea6d5ac98a371… --- diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml index cf08771326c..6850f53e9bf 100644 --- a/src/modules/siputils/doc/siputils_admin.xml +++ b/src/modules/siputils/doc/siputils_admin.xml @@ -1474,6 +1474,27 @@ if (!hdr_date_check("10")) { exit; } ... +</programlisting> + </example> + </section> + <section id="siputils.f.sip_parse_headers"> + <title> + <function moreinfo="none">sip_parse_headers()</function> + </title> + <para> + Return true if parsing (fast mode) all SIP headers is successful. + </para> + <para> + This function can be used from ANY_ROUTE. + </para> + <example> + <title><function>sip_parse_headers</function> usage</title> + <programlisting format="linespecific"> +... +if (sip_parse_headers()) { + ... +} +... </programlisting> </example> </section>

1 0

git:master:c04e2367: siputils: added function to parse headers
by Daniel-Constantin Mierla 26 Aug '24

26 Aug '24

Module: kamailio Branch: master Commit: c04e236721f9572a2655e39b8c5136bd23f95dd3 URL: https://github.com/kamailio/kamailio/commit/c04e236721f9572a2655e39b8c5136b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-26T15:00:07+02:00 siputils: added function to parse headers - could be useful after applying changes or replacing message buffer content --- Modified: src/modules/siputils/siputils.c --- Diff: https://github.com/kamailio/kamailio/commit/c04e236721f9572a2655e39b8c5136b… Patch: https://github.com/kamailio/kamailio/commit/c04e236721f9572a2655e39b8c5136b… --- diff --git a/src/modules/siputils/siputils.c b/src/modules/siputils/siputils.c index 372da25b041..ffbeacd00a5 100644 --- a/src/modules/siputils/siputils.c +++ b/src/modules/siputils/siputils.c @@ -111,6 +111,8 @@ static int w_contact_param_check(sip_msg_t *msg, char *pnparam, char *p2); static int w_hdr_date_check(sip_msg_t *msg, char *ptdiff, char *p2); +static int w_sip_parse_headers(sip_msg_t *msg, char *p1, char *p2); + /* Fixup functions to be defined later */ static int fixup_set_uri(void **param, int param_no); static int fixup_free_set_uri(void **param, int param_no); @@ -208,6 +210,7 @@ static cmd_export_t cmds[] = { fixup_spve_null, fixup_free_spve_null, ANY_ROUTE}, {"hdr_date_check", (cmd_function)w_hdr_date_check, 1, fixup_igp_null, fixup_free_igp_null, ANY_ROUTE}, + {"sip_parse_headers", (cmd_function)w_sip_parse_headers, 0, 0, 0, ANY_ROUTE}, {"bind_siputils", (cmd_function)bind_siputils, 1, 0, 0, 0}, @@ -605,6 +608,17 @@ static int w_hdr_date_check(sip_msg_t *msg, char *ptdiff, char *p2) return ki_hdr_date_check(msg, tdiff); } +/** + * + */ +static int w_sip_parse_headers(sip_msg_t *msg, char *p1, char *p2) +{ + if(parse_headers(msg, HDR_EOH_F, 0) < 0) { + return -1; + } + return 1; +} + /** * */

1 0

[kamailio/kamailio] During de-registration procedure, clean-up is not happening if contact has * value (Issue #3848)
by deepak-git-account 24 Aug '24

24 Aug '24

### Description If Deregistration contains * in a contact field as described in RFC-3261, then the Kamailio is not cleaning the in-memory and database record. And the next iteration is not working correctly. Use of the "*" Contact header field value allows a registering UA to remove all bindings associated with an address-of-record without knowing their precise values.  Step:1 REGISTER USER UE-A Step:2: Wait for the certain duration Step:3: De-REGISTER USER UE-A (Contact header is using *) Step:4: Repeat from the step-1 After step:3, clean-up is not happening. Record can be seen in SQL. Note: If de-register contains specific CONTACT header, then cleanup is happening. ### Troubleshooting #### Reproduction  #### Debugging Data  ``` (paste your debugging data here) ``` #### Log Messages  ``` May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: DEBUG: <script>: Contact headervalue: * May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=25 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=26 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: DEBUG: <script>: Full URI from sip:49404965200004@open-ims.test: sip:49404965200004@open-ims.test May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=33 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=34 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=36 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=37 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=38 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=39 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=46 a=16 n=if May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=43 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=54 a=16 n=if May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=46 a=25 n=is_present_hf May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=47 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=50 a=25 n=append_hf May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=54 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=55 a=25 n=pcscf_save_pending May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: ims_registrar_pcscf [save.c:318]: save_pending(): No contact headers May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: DEBUG: ims_registrar_pcscf [save.c:492]: save_pending(): Error saving pending contact May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=67 a=16 n=if May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=65 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[] c=[] l=0 a=12 n=seturi May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=71 a=26 n=xlog ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` kamailio -v version: kamailio 5.7.5 (x86_64/linux) 58499a-dirty flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: 58499a -dirty compiled on 11:23:55 May 8 2024 with gcc 9.4.0 ``` * **Operating System**:  ``` lsb_release -a No LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.6 LTS Release: 20.04 Codename: focal ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3848 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3848(a)github.com>

5 9

Re: [kamailio/kamailio] core: main - safety check for invalid syntax in listen parameter (9783c27)
by Henning Westerholt 23 Aug '24

23 Aug '24

Right now it will just ignore the additional slash and cut the string there: henning@app01:~/repositories/kamailio/src$ killall kamailio; ./kamailio -l udp:127.0.0.1:5060//te/st -f ../etc/test.cfg -Y /tmp/ -E -L modules Listening on udp: 127.0.0.1:5060 name te Aliases: udp: localhost:5060 I think the logic could be extended easily to allow e.g. a fourth field for the listen, when we need it. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd864b1de592/145744930(a)github.com>

1 0

Re: [kamailio/kamailio] core: main - safety check for invalid syntax in listen parameter (9783c27)
by Daniel-Constantin Mierla 23 Aug '24

23 Aug '24

Maybe a check should be added that there is no `/` is in the last field now/socket name, so in the future `/` can be used as separator for new fields. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd864b1de592/145744612(a)github.com>

1 0

← Newer
1
...
6
7
8
9
10
11
12
...
17
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2024