search

TECH7Fox / asterisk-hass-integration

Asterisk integration for Home Assistant
57 stars 13 forks source link

Fix race condition #64

Closed wolfwander closed 1 year ago

wolfwander commented 1 year ago

Fix a race condition that happens when Asterisk server is slower than Home Assistant to send the information abount extensions. The proposed changes calls the hass.config_entries.async_forward_entry_setup method only after the EndpointListComplete event is received. When the race condition happens, the integration can't parse the data of all extesions to Home Assistant entities.

TECH7Fox commented 1 year ago

Looks good. Thanks.

wolfwander commented 1 year ago

Thank you! Your integration helped me a lot for monitoring my Asterisk server.

Obter o Outlook para Androidhttps://aka.ms/AAb9ysg

From: Jordy Kuhne @.> Sent: Thursday, December 8, 2022 6:46:25 PM To: TECH7Fox/asterisk-hass-integration @.> Cc: wolfwander @.>; Author @.> Subject: Re: [TECH7Fox/asterisk-hass-integration] Fix race condition (PR #64)

Merged #64https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FTECH7Fox%2Fasterisk-hass-integration%2Fpull%2F64&data=05%7C01%7C%7C54331d0348544e17972008dad965a886%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638061327896692353%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=y9A0fcSTCZf1z4nbIw2FS3cfg2Y3lnOoZbqLwJIuo3U%3D&reserved=0 into main.

— Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FTECH7Fox%2Fasterisk-hass-integration%2Fpull%2F64%23event-7993256653&data=05%7C01%7C%7C54331d0348544e17972008dad965a886%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638061327896692353%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=3owMQ027MaHAekIDZyPWZwcqwPbS9ZnkrmJ7AQj6GM8%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGPXZ5TW5DWKOSXV26ZGSFTWMJJLDANCNFSM6AAAAAASYRMT5E&data=05%7C01%7C%7C54331d0348544e17972008dad965a886%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638061327896692353%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=38bv51%2BnYi5%2F0p1DF97Rg31lgS3Z7nqagkqbIkm%2BY8o%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

TECH7Fox commented 1 year ago

Good to hear! Hope you like it 🙂 Want to add some more functionality but don't have much time currently, so thanks for helping.