Dsmr-reader: AmbiguousTimeError causes excessive notifications

[GreyEarl]

This morning/night I received hundereds of daily consumption notifications through Pushover. All duplications. I even got a warning from Pushover that the designated channel exceeded the 7500 message limit.

It seems a message was sent at least every second.

[jeroenpeters1986]

I have got the same situation at hand. At 6.30 the counter of Prowl was over 2300.

I’m not sure if it’s relevant, but coming night has a daylight savings adjustment (nl: zomer/wintertijd).

I am using the mysql setup.

[tjla]

Same problem (using Pushover) here, >7000 messages,

[michielvisser]

Here as well, my API of Prowl stopped after 1000 messages with an error stating quota exceeded. I have not seen this issue before. BTW, at the moment I am stil running an older version of DSMR reader - 1.21.1.

Michiel

On 27 Oct 2018, at 07:53, tjla <[email protected][e-mail]> wrote:

Same problem (using Pushover) here, >7000 messages,

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dennissiemensma/dsmr-reader/issues/541#issuecomment-433593757, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOoXUdFDgcJlMvaQfe5-WkAJCrm7vKOgks5uo_S8gaJpZM4X9O0a.

[dennissiemensma]

Bedankt voor jullie meldingen. Ik ervaarde vannacht hetzelfde en vroeg me al af of ik de enigste was.

De oplossing voor nu is om in DSMR-reader notificaties uit te zetten, totdat ik er achter ben waar dit opeens vandaan komt en ik een fix heb om te voorkomen dat dit ooit weer gebeurt.

[dennissiemensma]

Het heeft inderdaad te maken met de wisseling zomertijd/wintertijd. Ik heb dit 5 maanden terug gerefactored toen NotifyMyAndroid ermee stopte en ik Pushover heb toegevoegd.

Daarbij vraag ik aan het systeem, ná het versturen van de notificatie, om voor mij de volgende dag om 2:00 te berekenen. Die gebruik ik als check-tijd om te bepalen of er vandaag al een notificatie is verstuurd. Dat gaat altijd goed, totdat die aankomende nacht tegenkomt, omdat die dat niet weet of ik 2:00 zomertijd of wintertijd bedoel. Python geeft daar simpelweg een foutmelding voor.
En doordat die eerst de notificatie verstuurt en daarna de 'check-tijd' een dag opschuift, komt die telkens een niet-aangepaste check-tijd tegen en blijft die het proberen. Normaal gesproken een wat lelijke situatie met niet zoveel gevolgen, maar nu wel omdat het daadwerkelijk een notificatie verstuurt naar iedereen en dat vrij hinderlijk is.

Ik zal kijken hoe ik dit goed oplos. Waarschijnlijk zet ik voor nu het notificatie-tijdstip simpelweg op 6:00, zodat dit niet meer kan gebeuren (en als het wel gebeurt dan kan iemand sneller reageren ipv wakker te worden met duizenden notificaties).

[dennissiemensma]

Ik heb een fix klaargezet voor de volgende release. Hij zou nu sowieso niet meer moeten crashen en daarnaast heb ik de notificatietijd naar 6:00 gezet ipv 2:00.

Meestal draai ik een versie die ik wil releasen eerst minimaal 24 uur zelf, om ervoor te zorgen dat backups en notificaties in ieder geval nog naar verwachting werken. Dat doe ik nu ook, dus ik release het niet eerder dan morgen.

Jullie kunnen tot die tijd:

• Notificaties uitschakelen in de admin-settings (makkelijkste is om puur even Notification service: op Disabled te zetten.

• Of switchen naar de development branch als je heel graag alsnog de notificatie wil, maar ik neem aan dat dat wel een dagje kan wachten. De applicatie werkt verder gewoon naar behoren.

[dennissiemensma]

Ik laat het ticket trouwens nog even openstaan bedenk ik me net, want er zijn vast meer die hier last van hebben en dit ticket mogelijk nog niet gezien hebben.

[GreyEarl]

Awesome! Bedankt voor de uitgebreide toelichting en het snelle handelen naar een oplossing!

[dennissiemensma]

Fix staat nu in nieuwere release.