- Use description found in https://sip-router.org/wiki/ref_manual/timers
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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 modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] PR should be backported to stable branches
- [ ] Tested changes locally
- [x] Related to issue #3939
#### Description
<!-- Describe your changes in detail -->
Fix documentation according to the description found in https://sip-router.org/wiki/ref_manual/timers. They better reflect the actual procedure.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4207
-- Commit Summary --
* tm: Update docs of fr_timer to match the implementation
-- File Changes --
M src/modules/tm/doc/params.xml (8)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4207.patchhttps://github.com/kamailio/kamailio/pull/4207.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4207
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4207(a)github.com>
Module: kamailio
Branch: master
Commit: 1195252256858dc4419c179e5b72541018f5feab
URL: https://github.com/kamailio/kamailio/commit/1195252256858dc4419c179e5b72541…
Author: Xenofon Karamanos <xk(a)gilawa.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2025-04-11T13:29:33+02:00
tm: Update docs of fr_timer to match the implementation
- Use description found in https://sip-router.org/wiki/ref_manual/timers
---
Modified: src/modules/tm/doc/params.xml
---
Diff: https://github.com/kamailio/kamailio/commit/1195252256858dc4419c179e5b72541…
Patch: https://github.com/kamailio/kamailio/commit/1195252256858dc4419c179e5b72541…
---
diff --git a/src/modules/tm/doc/params.xml b/src/modules/tm/doc/params.xml
index 1dd77f1df3f..31c7c55b9ef 100644
--- a/src/modules/tm/doc/params.xml
+++ b/src/modules/tm/doc/params.xml
@@ -18,8 +18,12 @@
<section id="tm.p.fr_timer">
<title><varname>fr_timer</varname> (integer)</title>
<para>
- Timer which hits if no final reply for a request or ACK for a
- negative INVITE reply arrives (in milliseconds).
+ This timer is used for all SIP requests. It hits if no reply for an
+ request INVITE request or other request has been received
+ within (F in milliseconds). If a provisional reply is received for an
+ INVITE (any 1xx), then the fr_inv_timer will be used instead. And if
+ no replies (at all) for an INVITE are received before fr_timer hits,
+ the transaction is terminated with a 408 in failure route.
</para>
<para>
Default value is 30000 ms (30 seconds).
Hampas-J created an issue (kamailio/kamailio#4211)
Hi,
I'm using "version: kamailio 6.0.0-dev2 (x86_64/linux) 0da3eb" to setup IMS and configured modparam("ims_ipsec_pcscf", "ipsec_preferred_ealg", "aes-cbc") to encrypt ESP payload, I can make a call, but I see the ealg is still null in SIP 401 Unauthorized message and ESP payload is not encrypted:
<img width="211" alt="Image" src="https://github.com/user-attachments/assets/f0d75ff0-1e40-429a-9511-da30cf63…" />
Is there any configuration missing?
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4211
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4211(a)github.com>
Module: kamailio
Branch: master
Commit: 92e99afa95df80c762ce60f3b9eca7b7936c74b9
URL: https://github.com/kamailio/kamailio/commit/92e99afa95df80c762ce60f3b9eca7b…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2025-04-11T10:01:17+02:00
modules: readme files regenerated - ims_registrar_scscf ... [skip ci]
---
Modified: src/modules/ims_registrar_scscf/README
---
Diff: https://github.com/kamailio/kamailio/commit/92e99afa95df80c762ce60f3b9eca7b…
Patch: https://github.com/kamailio/kamailio/commit/92e99afa95df80c762ce60f3b9eca7b…
---
diff --git a/src/modules/ims_registrar_scscf/README b/src/modules/ims_registrar_scscf/README
index 100ea1e657c..5ef62cbb0e1 100644
--- a/src/modules/ims_registrar_scscf/README
+++ b/src/modules/ims_registrar_scscf/README
@@ -682,7 +682,7 @@ switch ($retcode) {
4.8. reg_fetch_contacts(domain, uri, profile)
The function fetches the contacts for 'uri' from table 'domain' to
- pseudo-variable $imssulc(profile) [imssulc = ims scscf ulc].
+ pseudo-variable $ulc(profile) [ulc = ims scscf ulc].
Meaning of the parameters is as follows:
* domain - Name of table that should be used for the lookup of
@@ -690,8 +690,8 @@ switch ($retcode) {
* uri - The SIP URI address of the user which to fetch the contact
addresses for. It can contain pseudo-variables that are evaluated
at runtime.
- * profile - Name of $imssulc pseudo-variable profile that will store
- the fetched contacts. It is a static string.
+ * profile - Name of $ulc pseudo-variable profile that will store the
+ fetched contacts. It is a static string.
This function can be used in REQUEST_ROUTE, FAILURE_ROUTE
@@ -709,7 +709,7 @@ reg_fetch_contacts("location", "sip:user@kamailio.org", "caller");
data in that profile.
Meaning of the parameters is as follows:
- * profile - Name of $imssulc pseudo-variable profile that stores the
+ * profile - Name of $ulc pseudo-variable profile that stores the
contacts. It is a static string.
This function can be used in REQUEST_ROUTE, FAILURE_ROUTE
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4166
-- Commit Summary --
* ims_registrar_scscf: fix reg_fetch_contacts call
* ims_registrar_scscf: fix documentation
-- File Changes --
M src/modules/ims_registrar_scscf/doc/ims_registrar_scscf_admin.xml (6)
M src/modules/ims_registrar_scscf/regpv.c (25)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4166.patchhttps://github.com/kamailio/kamailio/pull/4166.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4166
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4166(a)github.com>
Sorry, to be annoying, but I would prefer to keep here the "-1", as we are not having any memory allocated, and to stay in sync with the main registrar module.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/commit/3f942b0d9519fdc0686569ec7e02f65…
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/commit/3f942b0d9519fdc0686569ec7e02f65feaadab27/155173722(a)github.com>