
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

</head>

<body dir="ltr">

<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr">

<p></p>

<div>Hi,</div>

<div><br>

</div>

<div>the description of "expires_offset" says "The value in seconds that should be subtracted from the expires value when sending a 200OK for a publish"</div>

<div><br>

</div>

<div>but this value is used for subscriptions and AFAICT its behaviour is not correct.</div>

<div>when checking expired subscriptions it subtracts the offset from current time instead of adding it.</div>

<div><br>

</div>

<div>in update_db_subs_timer_dbonly</div>

<div>...</div>

<div><span class="Apple-tab-span" style="white-space:pre"></span>qcols[0]= &str_expires_col;</div>

<div><span class="Apple-tab-span" style="white-space:pre"></span>qvals[0].type = DB1_INT;</div>

<div><span class="Apple-tab-span" style="white-space:pre"></span>qvals[0].nul = 0;</div>

<div><span class="Apple-tab-span" style="white-space:pre"></span>qvals[0].val.int_val= (int)time(NULL) - expires_offset;</div>

<div><span class="Apple-tab-span" style="white-space:pre"></span>qops[0]= OP_LT;</div>

<div>...</div>

<div><br>

</div>

<div>i think the behavior should be to subtract the value when sending the NOTIFY but add (or not use it) when checking for expired records.</div>

<div>if anyone thinks this shouldn't be like this, please comment.</div>

<div>i'll try to create a PR for this if no comments against this.</div>

<div><br>

</div>

<div>Thanks</div>

<div><br>

</div>

<br>

<p></p>

</div>

</body>

</html>