Error after session active > 24 hours

[sleuth255]

Error after session active > 24 hours

[Apollon77]

No code is correct.

If you set the Intervall to 0 then no auto refresh is done. Else it will be used it set to 7days if it has a falsy value (false, null, undefined)

[Apollon77]

you could add the respinse code to the Exception so that we could see what exactly the response is about. But if amazon is returning this ... could Isee your code somewhere? I personally do not experience such an effect

[sleuth255]

yes, I've done that and am waiting for it to fail again (should be in about 15 hours).

I'm actually using your alexa-remote.js code as-is as part of a node.js web wrapper that implements the speak function. It's targeted for use as an Alexa TTS API by home automation systems such as Home Control Assistant. https://github.com/sleuth255/alexa-speaks

Your code is in the public folder. Problem occurs whether useWsMqtt is true or false. Interestingly, when the problem is occurring, your getDevices function still works fine. It is only the speech function that is failing.

[Apollon77]

Why you do not use the lib as dependency but copy the code?!
Ok ... can it be that there is a time where too many requests are send too fast and amazon rate limiting kicks in?

[sleuth255]

I wouldn't think so. It fails even if I only send one command (see the logs above). I've had to work out how alexa-remote operates by looking at the code pretty much, so it was just easier to have it handy. Maybe I'm not setting everything up correctly? I'm directly using sendSequenceCommand with a "speak" function, init, and getDevices. from your code currently.

[sleuth255]

Hmmm... its still up. The only change I made was fix a problem with the refresh cookie interval. I now have the cookie being refreshed every 12 hours. Maybe the en-US locale/site requires a more frequent cookie refresh cycle than once every 7 days? Still, this doesn't explain why getDevices was still functional but the speak function was failing. I'll keep it running and if it lasts another day, I'll close the issue.

[sleuth255]

more on the bug which is that the default cookie timeout wasn't being set correctly on lines 95/96 of alexa-remote because the option name is being transposed to refreshCookieInterval.

[Apollon77]

Last bug I thought was fixed already ... maybe your code relies on an old version of Alexa cookie lib and remote lib because you hard Code all deps.

[sleuth255]

shouldn't those two lines of code say this:

    if (this._options.cookieRefreshInterval === 0) {
        this._options.cookieRefreshInterval = this._options.cookieRefreshInterval || 7*24*60*60*1000; // Auto Refresh after 7 days

??

[sleuth255]

so

   if (this._options.cookieRefreshInterval !== 0) {
       this._options.cookieRefreshInterval = this._options.cookieRefreshInterval || 7*24*60*60*1000; // Auto Refresh after 7 days

is correct then. That's how I have it currently. changed from this:

    if (this._options.refreshCookieInterval !== 0) {
        this._options.refreshCookieInterval = this._options.refreshCookieInterval || 7*24*60*60*1000; // Auto Refresh after 7 days

[sleuth255]

I'll pull your library in as a dependency. It looks like my setting it to 12x60x60x1000 coupled with the above change has allowed the server to run for more 32+ hours and counting so far.

[Apollon77]

Hm ... it is very strange that it should need refreshs that often.

[sleuth255]

Yea, it failed again after about 35 hours. Oddly, no error code was displayed even after I changed the error to display it. I’ll fork/do pull request when I uncover the cause. From: Ingo Fischer [mailto:[email protected]] Sent: Tuesday, December 10, 2019 11:16 PM To: Apollon77/alexa-remote <[email protected]> Cc: sleuth255 <[email protected]>; State change <[email protected]> Subject: Re: [Apollon77/alexa-remote] Error after session active > 24 hours (#57) Hm ... it is very strange that it should need refreshs that often. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#57?email_source=notifications&email_token=ACNXU2QCBQC3YNK62FINWRTQYBZSLA5CNFSM4JYJ2ERKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGR5AUY#issuecomment-564383827> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACNXU2VT5NTTDMDZNK2Y57TQYBZSLANCNFSM4JYJ2ERA> . <https://github.com/notifications/beacon/ACNXU2VCANRYOY6ZS5UW5WDQYBZSLA5CNFSM4JYJ2ERKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGR5AUY.gif>