<p>Maybe the regression is in this commit</p>
<pre><code>commit e712d32f9304f95b5b19b26905546b4af700298f
Author: Daniel-Constantin Mierla <miconda@gmail.com>
Date: Thu Mar 2 16:40:41 2017 +0100
topos_redis: update fields and reduce expires for termintated dialogs
diff --git a/src/modules/topos_redis/topos_redis_storage.c b/src/modules/topos_redis/topos_redis_storage.c
index 08ead91..100e823 100644
--- a/src/modules/topos_redis/topos_redis_storage.c
+++ b/src/modules/topos_redis/topos_redis_storage.c
@@ -267,7 +267,7 @@ int tps_redis_insert_dialog(tps_data_t *td)
rrpl = _tps_redis_api.exec_argv(rsrv, argc, (const char **)argv, argvlen);
if(rrpl==NULL) {
- LM_ERR("failed to execute expireredis command\n");
+ LM_ERR("failed to execute expire redis command\n");
if(rsrv->ctxRedis->err) {
LM_ERR("redis error: %s\n", rsrv->ctxRedis->errstr);
}
@@ -386,7 +386,7 @@ int tps_redis_insert_branch(tps_data_t *td)
argvlen[argc] = rkey.len;
argc++;
- lval = (unsigned long)_tps_api.get_dialog_expire();
+ lval = (unsigned long)_tps_api.get_branch_expire();
if(lval==0) {
return 0;
}
</code></pre>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/issues/1848#issuecomment-462829275">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZR9PV3_pn4At5OmONE1Q1pX92oHpks5vMut7gaJpZM4a0o_2">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36Zd-eWfe4zGzQeA-wvKw_uqfVKdxAks5vMut7gaJpZM4a0o_2.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@jchavanton in #1848: \r\nMaybe the regression is in this commit\r\n\r\n```\r\ncommit e712d32f9304f95b5b19b26905546b4af700298f\r\nAuthor: Daniel-Constantin Mierla \u003cmiconda@gmail.com\u003e\r\nDate: Thu Mar 2 16:40:41 2017 +0100\r\n\r\n topos_redis: update fields and reduce expires for termintated dialogs\r\n\r\ndiff --git a/src/modules/topos_redis/topos_redis_storage.c b/src/modules/topos_redis/topos_redis_storage.c\r\nindex 08ead91..100e823 100644\r\n--- a/src/modules/topos_redis/topos_redis_storage.c\r\n+++ b/src/modules/topos_redis/topos_redis_storage.c\r\n@@ -267,7 +267,7 @@ int tps_redis_insert_dialog(tps_data_t *td)\r\n \r\n rrpl = _tps_redis_api.exec_argv(rsrv, argc, (const char **)argv, argvlen);\r\n if(rrpl==NULL) {\r\n- LM_ERR(\"failed to execute expireredis command\\n\");\r\n+ LM_ERR(\"failed to execute expire redis command\\n\");\r\n if(rsrv-\u003ectxRedis-\u003eerr) {\r\n LM_ERR(\"redis error: %s\\n\", rsrv-\u003ectxRedis-\u003eerrstr);\r\n }\r\n@@ -386,7 +386,7 @@ int tps_redis_insert_branch(tps_data_t *td)\r\n argvlen[argc] = rkey.len;\r\n argc++;\r\n \r\n- lval = (unsigned long)_tps_api.get_dialog_expire();\r\n+ lval = (unsigned long)_tps_api.get_branch_expire();\r\n if(lval==0) {\r\n return 0;\r\n }\r\n```"}],"action":{"name":"View Issue","url":"https://github.com/kamailio/kamailio/issues/1848#issuecomment-462829275"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/issues/1848#issuecomment-462829275",
"url": "https://github.com/kamailio/kamailio/issues/1848#issuecomment-462829275",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>