-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
Octoprint integration stopped working with basic auth #128264
Comments
Hey there @rfleming71, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) octoprint documentation |
Can you tell me which version of octoprint you are on? I don't believe anything has changed with the integration in many releases. |
#126961 |
Found bug fix in yarn aio-libs/yarl#880 (comment), now it’s impossible to use basic auth in url, but it’s not recommended to just leave octoprint open. Maybe add auth to plugin? |
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. |
The problem
After installing update, octoprint showed 0 entries, after checking the logs found this message
Error while setting up octoprint platform for camera Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 361, in _async_setup_platform await asyncio.shield(awaitable) File "/usr/src/homeassistant/homeassistant/components/octoprint/camera.py", line 40, in async_setup_entry OctoprintCamera( File "/usr/src/homeassistant/homeassistant/components/octoprint/camera.py", line 66, in __init__ device_info=coordinator.device_info, ^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/components/octoprint/coordinator.py", line 82, in device_info configuration_url = URL.build( ^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 385, in build netloc = cls._make_netloc( ^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 1057, in _make_netloc ret = cls._encode_host(host) ^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 1038, in _encode_host _host_validate(host) File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 1607, in _host_validate raise ValueError( ValueError: Host 'user:pass@example.com' cannot contain ':' (at position 6), if the value includes a username or password, use 'authority' instead of 'host'
basic auth is recommend way of exposing octoprint, maybe add basic auth to integration parameters, like other apps
What version of Home Assistant Core has the issue?
core-2024.10.2
What was the last working version of Home Assistant Core?
core-2024.8.0
What type of installation are you running?
Home Assistant OS
Integration causing the issue
octoprint
Link to integration documentation on our website
https://www.home-assistant.io/integrations/octoprint
Diagnostics information
No response
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: