Would love to get this merged, but didn't succeed testing it successfully on my
machine. Maybe I did something wrong? A second pair of eyes would be highly
appreciated.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3766#issuecomment-2172759113
> ERROR: topos [tps_storage.c:1407]: tps_db_load_dialog(): invalid dlg uuid
> provided ,This error was noticed before the last commit
> [[9769806](https://github.com/kamailio/kamailio/commit/9769806b96148a2ede4612605c6d4af047c137f7)].
> After this update it was fine. I have tested with more
> I tested it for REGISTER, and it seems to break if the registrar sends e.g. a
> 4xx with no contact back, causing the log line
>
> ```
> ERROR: topos [tps_storage.c:1407]: tps_db_load_dialog(): invalid dlg uuid
> provided
> ```
>
> Is this something you've experienced, @toharish ?
Thinking
I tested it for REGISTER, and it seems to break if the registrar sends e.g. a
4xx with no contact back, causing the log line
```
ERROR: topos [tps_storage.c:1407]: tps_db_load_dialog(): invalid dlg uuid
provided
```
Is this something you've experienced, @toharish ?
--
Reply to this email
Interested in that one, as I by chance have an immediate need for it. Will test
it today.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3766#issuecomment-1991461000
You are receiving this because you are subscribed to this thread.
Message ID:
Module: kamailio
Branch: master
Commit: 2bb8b5689c3064bd0ad1da80bdcfa36f29d2b8e3
URL:
https://github.com/kamailio/kamailio/commit/2bb8b5689c3064bd0ad1da80bdcfa36f29d2b8e3
Author: Andreas Granig
Committer: Andreas Granig
Date: 2022-10-13T17:42:47+02:00
mqtt: Add tls_alpn module option
Merged #3261 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3261#event-7582729588
You are receiving this because you are subscribed to this thread.
Message ID: ___
Kamailio (SER) -
@agranig commented on this pull request.
> @@ -324,6 +324,27 @@ modparam("mqtt", "ca_path", "/etc/ssl/certs")
...
modparam("mqtt", "tls_method", "tlsv1.3")
+...
+
+
+
+
+ tls_alpn (str)
+
+Used
@agranig commented on this pull request.
> @@ -324,6 +324,27 @@ modparam("mqtt", "ca_path", "/etc/ssl/certs")
...
modparam("mqtt", "tls_method", "tlsv1.3")
+...
+
+
+
+
+ tls_alpn (str)
+
+Used
@agranig commented on this pull request.
> @@ -324,6 +324,27 @@ modparam("mqtt", "ca_path", "/etc/ssl/certs")
...
modparam("mqtt", "tls_method", "tlsv1.3")
+...
+
+
+
+
+ tls_alpn (str)
+
+Used
@agranig commented on this pull request.
> @@ -324,6 +324,27 @@ modparam("mqtt", "ca_path", "/etc/ssl/certs")
...
modparam("mqtt", "tls_method", "tlsv1.3")
+...
+
+
+
+
+ tls_alpn (str)
+
+Used
@agranig pushed 1 commit.
bf5cf64f1daf8f0c8062e32d116c5da357fe516d mqtt: Add tls_alpn module option
--
View it on GitHub:
https://github.com/kamailio/kamailio/pull/3261/files/43981e28e4fa48f6f6f69af1594f671742d1e4cf..bf5cf64f1daf8f0c8062e32d116c5da357fe516d
You are receiving this because you
@agranig commented on this pull request.
> @@ -324,6 +324,27 @@ modparam("mqtt", "ca_path", "/etc/ssl/certs")
...
modparam("mqtt", "tls_method", "tlsv1.3")
+...
+
+
+
+
+ tls_alpn (str)
+
+Used
This is used when a service (like AWS IoT Core) uses one TLS port for multiple
services (like https and mqtt), so you have to set the ALPN to mqtt
to be able to connect kamailio.
!-- Kamailio Pull Request Template --
!--
IMPORTANT:
- for detailed contributing guidelines, read:
Thanks for the patch, looks good! I'll take care merging it.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Hey, unfortunately it doesn’t support it yet, and it’s not clear yet when I can
address this, as I don’t have immediate needs for it.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
agranig approved this pull request.
+2, looks good to me
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Hey, yes, absolutely interested, just didn’t get around following up on this
further!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Module: kamailio
Branch: master
Commit: cc6865cd7d0fc48709acd7685d315eb5575e45bd
URL:
https://github.com/kamailio/kamailio/commit/cc6865cd7d0fc48709acd7685d315eb5575e45bd
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-04-11T17
agranig approved this pull request.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1495#pullrequestreview-110413756___
Kamailio (SER)
Module: kamailio
Branch: master
Commit: 0729d9fc2cde7caf3e21b3e609cd26b24e38c19b
URL:
https://github.com/kamailio/kamailio/commit/0729d9fc2cde7caf3e21b3e609cd26b24e38c19b
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-04-05T11
Module: kamailio
Branch: master
Commit: 22bd6ca04dd20951a20aca77ec9a8cb46cdf73f9
URL:
https://github.com/kamailio/kamailio/commit/22bd6ca04dd20951a20aca77ec9a8cb46cdf73f9
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-03-19T17
Module: kamailio
Branch: master
Commit: c9f2aa71b2e0d4a4f71b3da92f97306e86f93eea
URL:
https://github.com/kamailio/kamailio/commit/c9f2aa71b2e0d4a4f71b3da92f97306e86f93eea
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-03-08T15
Module: kamailio
Branch: master
Commit: b62d623578737a85dd18cacc66f4e71232b463aa
URL:
https://github.com/kamailio/kamailio/commit/b62d623578737a85dd18cacc66f4e71232b463aa
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-03-01T09
Module: kamailio
Branch: master
Commit: 62d7c838b5bcef04c3445b9eb9c6d73402f2b02f
URL:
https://github.com/kamailio/kamailio/commit/62d7c838b5bcef04c3445b9eb9c6d73402f2b02f
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-02-27T17
Try redis://password@host:port/db and let me know if it works for you.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Module: kamailio
Branch: master
Commit: fd9fe3a677cde97dbf00ef965c45e5b6a495edf7
URL:
https://github.com/kamailio/kamailio/commit/fd9fe3a677cde97dbf00ef965c45e5b6a495edf7
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-02-20T11
Module: kamailio
Branch: master
Commit: e727ddcef5113c5235d8d2dde9da86a8c2d2b4bb
URL:
https://github.com/kamailio/kamailio/commit/e727ddcef5113c5235d8d2dde9da86a8c2d2b4bb
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-02-14T15
Merged #1432.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1432#event-1473375435___
Kamailio (SER) - Development Mailing List
Module: kamailio
Branch: master
Commit: 56ad14236c02af1c82e3bdbcf1f85bb6f0efd728
URL:
https://github.com/kamailio/kamailio/commit/56ad14236c02af1c82e3bdbcf1f85bb6f0efd728
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-02-13T16
Module: kamailio
Branch: master
Commit: 53e746b5c527fa542e4b2b7353af5a3b4042d7d2
URL:
https://github.com/kamailio/kamailio/commit/53e746b5c527fa542e4b2b7353af5a3b4042d7d2
Author: Andreas Granig <agra...@sipwise.com>
Committer: Andreas Granig <agra...@sipwise.com>
Date: 2018-02-08T15
Hi @miconda, I've improved the module with the suggestions you made:
* utilize xml schema definitions and load them from files to avoid having to
define them as mod params
* allow multiple keys mod params definitions to make config more readable
* reconnect improvements
* documentation fixes and
@agranig pushed 1 commit.
609c350 db_redis: Use schema files and improve keys def
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@agranig pushed 1 commit.
2378fb9 db_redis: Fix missed freeing redis reply objects
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@agranig pushed 1 commit.
dcb8611 db_redis: Cleanups and fixes
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/kamailio/kamailio/pull/1432/files/aab86a5595b4b6294d9a1073321d623520f3fc0c..dcb86110b481a32071d1b87a7c6066dbc8ce810c
JFYI, there is another generic implementation available as of today at
https://github.com/agranig/kamailio/tree/agranig/db_redis/src/modules/db_redis.
It's currently undergoing internal reviews, I'll send a pull request in a
couple of days. Compared to Surendra's implementation, it's supposed
36 matches
Mail list logo