Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Probable issue if module runs for 50 days without being reset #11

Open
reivaxy opened this issue Aug 29, 2016 · 1 comment
Open

Probable issue if module runs for 50 days without being reset #11

reivaxy opened this issue Aug 29, 2016 · 1 comment

Comments

@reivaxy
Copy link
Owner

reivaxy commented Aug 29, 2016

The anti-sms-flood interval for each number that receives an alert stores the value returned by millis() when sending an SMS alert.
Next time sending alerts is checked, it compares the elapsed time between this value and the current millis() value.
But millis() resets every 50 days approximately. The stored value, before the reset, will then be bigger than any millis() returned value for the next 50 days => no more alerts sent.

@reivaxy reivaxy added the bug label Aug 29, 2016
@reivaxy
Copy link
Owner Author

reivaxy commented Aug 30, 2016

Needs more investigation, a while ago I thought it would not be an issue (and wrote it down!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant