Dynamic session timeout

[Alan DeKok]

On Nov 10, 2018, at 12:57 PM, Sam T <givemesam at gmail.com> wrote:
> Is it necessary to use v3 and run the script you recently sent?

  Yes,

> Do the
> "exper parsing improvements" you mentioned in v3 necessary in the use of
> your script?

  The patch fixes parsing errors.  Without the patch, it won't work.

> (I'm not sure we can change the server version as we are on an aging system
> with many working bits and customizations over the years)
> 
> My team came up with this 2query SQL approach that does give the expected
> results, it has multiple queries, and I wanted to ask your opinion to
> improve it
> 
>        if ("%{sql:SELECT
> TIMESTAMPDIFF(SECOND,(NOW()),'%{reply:WISPr-Session-Terminate-Time}')#}" <=
> "%{reply:Session-Timeout}") {
>        update reply {
> Session-Timeout := "%{sql:SELECT
> TIMESTAMPDIFF(SECOND,(NOW()),'%{reply:WISPr-Session-Terminate-Time}')}"
>        }
>        }

  If it works, it's fine.

  Alan DeKok.