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

BREAKING CHANGE in HA 2021.12 #179

Closed
bafplus opened this issue Dec 6, 2021 · 13 comments
Closed

BREAKING CHANGE in HA 2021.12 #179

bafplus opened this issue Dec 6, 2021 · 13 comments
Labels
duplicate This issue or pull request already exists

Comments

@bafplus
Copy link

bafplus commented Dec 6, 2021

Although HA is giving suggestions for months WebRTC is still not updated to stay working with Home Assistant. From HA 2021.12 this will be a BREAKING CHANGE:

Error during setup of component webrtc
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/setup.py", line 229, in _async_setup_component
result = await task
File "/config/custom_components/webrtc/init.py", line 75, in async_setup
utils.register_static_path(hass.http.app, url_path, path)
File "/config/custom_components/webrtc/utils.py", line 74, in register_static_path
app'allow_cors'
File "/usr/local/lib/python3.9/site-packages/aiohttp/web_app.py", line 186, in getitem
return self._state[key]
KeyError: 'allow_cors'

@zvldz
Copy link

zvldz commented Dec 6, 2021

update to master

@bafplus
Copy link
Author

bafplus commented Dec 6, 2021 via email

@zvldz
Copy link

zvldz commented Dec 6, 2021

Changes in the master version of the component, no release yet. You can install the master version via hacs

@bafplus
Copy link
Author

bafplus commented Dec 6, 2021

You cant, HACS says:

Repository 'alexxit/webrtc' exists in the store.

@sean-m-sullivan
Copy link

Took me a moment in hacs when adding repository
image

@sean-m-sullivan
Copy link

however I just switched to master and have the same issue with allow_cors

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 7, 2021

Duplicate

@AlexxIT AlexxIT closed this as completed Dec 7, 2021
@AlexxIT AlexxIT added the duplicate This issue or pull request already exists label Dec 7, 2021
@bafplus
Copy link
Author

bafplus commented Dec 7, 2021

Switching to Master did fix it, but i would suggest to push a new version else everyones HA install will fail on WebRTC next saturday.

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 7, 2021

@bafplus if you see this error, you have not installed the master version

@bafplus
Copy link
Author

bafplus commented Dec 7, 2021

@bafplus if you see this error, you have not installed the master version

As said above, master did fix it. Yet i highly suggest to push a version update to avoid mass failures by users on next HA update next saturday :-)

@gryphon-wg
Copy link

2AlexxIT
Installed master version, 2 time reboot, but still no luck

HACS: " version downloaded 0e45d1"
LOGS: Error during setup of component webrtc
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/setup.py", line 229, in _async_setup_component
result = await task
File "/config/custom_components/webrtc/init.py", line 75, in async_setup
utils.register_static_path(hass.http.app, url_path, path)
File "/config/custom_components/webrtc/utils.py", line 74, in register_static_path
app'allow_cors'
File "/usr/local/lib/python3.9/site-packages/aiohttp/web_app.py", line 186, in getitem
return self._state[key]
KeyError: 'allow_cors'
Integrations: Not loaded

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 12, 2021

@gryphon-wg as you can see, line 74 don't have this code. So you haven't installed the master version

if 'allow_all_cors' in app:

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 13, 2021

#156

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

5 participants