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

Fix call to Esios Server since 2021-11-02 (#27) #28

Merged
merged 5 commits into from
Nov 3, 2021
Merged

Conversation

azogue
Copy link
Owner

@azogue azogue commented Nov 3, 2021

closes #27

Changes:

  • Fix Esios request returning a 403 status code since 2021-11-02, by:
    • just adding an 'User-Agent' to request headers, if aiohttp==3.7.4.post0
    • or upgrading to aiohttp==3.8.0, where it is not needed and the original request works like before
  • Add better error logging for this 'forbidden' error if reappears in the future
  • Update deps, requiring holidays>0.11.1

 since 2021-11-02, by:
 - just adding an 'User-Agent' to request headers, if `aiohttp==3.7.4.post0`
 - or upgrading to `aiohttp==3.8.0`, where it is not needed and the
   original request works like before.

And add better error logging for this 'forbidden' error if reappears
@azogue azogue merged commit da7c138 into master Nov 3, 2021
azogue added a commit to azogue/home-assistant that referenced this pull request Nov 3, 2021
happening because some config change in the ESIOS API server,
solved with a version patch in aiopvpc
(details in azogue/aiopvpc#28)
MartinHjelmare pushed a commit to home-assistant/core that referenced this pull request Nov 4, 2021
happening because some config change in the ESIOS API server,
solved with a version patch in aiopvpc
(details in azogue/aiopvpc#28)
@azogue azogue deleted the fix/esios-request branch November 4, 2021 08:07
balloob pushed a commit to home-assistant/core that referenced this pull request Nov 5, 2021
happening because some config change in the ESIOS API server,
solved with a version patch in aiopvpc
(details in azogue/aiopvpc#28)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No funciona la integración con HA desde el 2 de noviembre
1 participant