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

Обработка кода ошибки 429 с сервера #26

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

DrRamm
Copy link

@DrRamm DrRamm commented Nov 3, 2024

No description provided.

@DrRamm
Copy link
Author

DrRamm commented Nov 3, 2024

Самое плохое, что у меня внезапно устройство определилось. И я не знаю, это мои измнения для 429 или сервак ожил. Ну хотя бы код запустился.

Кто-то может мне пояснить за self.refresh()? Я грепом прошёлся и не нашёл, поэтому переделал на self.login(True)

@de-andrei
Copy link

Исправил вызов метода refresh - заработало. Спасибо

@DrRamm
Copy link
Author

DrRamm commented Nov 6, 2024

@de-andrei я подозреваю, что на самом деле вы просто ребутнули плагин и звезды сошлись. У меня даже со всеми доработками вылетела 429 при ребуте HA. Вырубил плагин, подождал часок-другой, врубил -- и всё завелось

Но теперь хотя бы ретраи есть и пишет внятно что происходит.

И вот для размышления: при 429 в HA родная прила эво нормально работает.

@de-andrei
Copy link

de-andrei commented Nov 7, 2024

@DrRamm, да Вы оказались правы. После рестарта ха, опять недоступно. Плюнул, перекинул в ха из удя

@tumb1er
Copy link

tumb1er commented Nov 11, 2024

@DrRamm, да Вы оказались правы. После рестарта ха, опять недоступно. Плюнул, перекинул в ха из удя

Проблемы тоже после рестарта: похоже, авторизация не переживает рестарта HA и выполняется сразу после старта невзирая на лимиты. В итоге, "монотонность" нарушается и возникает 429. Я бы попробовал токен сохранять куда-нибудь и загружать его при старте вместо получения с нуля.

@SeRG1k17
Copy link

SeRG1k17 commented Dec 10, 2024

@DrRamm Добавил сегодня интеграцию в НА. Не видит устройств, сразу вижу 429, удалил приложение evo, не помогает. Если МР решает проблему, могу потестить на форке

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.

4 participants