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

Buienradar API bron/foutafhandeling verbeteren #755

Closed
mrvanes opened this issue Oct 24, 2019 · 10 comments
Closed

Buienradar API bron/foutafhandeling verbeteren #755

mrvanes opened this issue Oct 24, 2019 · 10 comments
Labels
Milestone

Comments

@mrvanes
Copy link

mrvanes commented Oct 24, 2019

Wat gebruik je?

  • kersverse DSMR pull

Sinds gisteravond 17.00 triggert de Track weather plugin ~1000 api.buienradar.nl DNS requests per 10 minuten, wat opviel in m'n Pi-hole dashboard. Disablen van de Track Weather plugin stopt de requests.
image

@tiki1980
Copy link

Ik heb dit ook gezien idd, per seconde ongeveer één uitvraag.

image

@mrvanes
Copy link
Author

mrvanes commented Oct 24, 2019

Het lijkt erop dat het API request mislukt, maar dat de plugin nogal agressief door blijft vragen. Dat heb ik verder niet onderzocht.

@vincedh
Copy link

vincedh commented Oct 24, 2019

Hier hetzelfde probleem inderdaad... Voorlopig de module maar uitgezet.

@dennissiemensma
Copy link
Member

Bedankt voor jullie melding. Ik zie hier nu hetzelfde: Failed to read Buienradar API.

Het is uiteraard niet de bedoeling dat die het zovaak opnieuw probeert. Zoiets was al eerder gemeld in #399 en ik dacht dat ik er toen ook iets aan had gedaan. Ik onderzoek eerst even de oorzaak.

@dennissiemensma
Copy link
Member

dennissiemensma commented Oct 24, 2019

De API zelf lijkt kapot of veranderd: https://api.buienradar.nl/

Er komt wel een HTTP 200 OK terug, maar geen inhoud. Ik weet niet of er een storing is, of dat RTL de API de nek om heeft gedraaid.

@dennissiemensma
Copy link
Member

Ik zie in https://www.buienradar.nl/overbuienradar/gratis-weerdata geen melding van wijzigingen, maar wel een referentie naar https://data.buienradar.nl/2.0/feed/json met soortgelijke gegevens.

Wellicht is dat een oplossing.

@dennissiemensma
Copy link
Member

dennissiemensma commented Oct 24, 2019

Ik ben er ook achter waarom DSMR-reader het elke seconde opnieuw probeert.

Er zit een check in om bij onbereikbare API 5 minuten te wachten. Hetzelfde geldt voor onjuiste data. Helaas klapt die er uit op een derde scenario: geen data. Dus ik zal het herschrijven naar een nog betere check.

@dennissiemensma dennissiemensma changed the title Track weather API hammered DNS server Buienradar API bron/foutafhandeling verbeteren Oct 24, 2019
@dennissiemensma dennissiemensma added this to the 2.9.0 milestone Oct 24, 2019
@dennissiemensma
Copy link
Member

Ik heb zowel de foutafhandeling verbeterd als de nieuwe API van buienradar ingebouwd. Gaat mee met volgende release.

@dennissiemensma
Copy link
Member

Even dit issue pinnen zodat helder is dat het opgelost wordt.

@dennissiemensma
Copy link
Member

Uitgebracht in v2.9.0.

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

No branches or pull requests

4 participants