-
Notifications
You must be signed in to change notification settings - Fork 384
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
Add Home Assistant Fusion UI #1185
Conversation
89ac8da
to
c62ad5c
Compare
Nice :) |
And here is another fantastic submission of @joaovictor-local 🎉 Thank you so much for your hard work on bringing new apps to umbrelOS! I have some suggestions:
Great work on reducing the user permissions to |
@sharknoon Thank you @sharknoon :)
app_proxy:
environment:
APP_HOST: home-assistant-fusion_web_1
APP_PORT: 5050
PROXY_AUTH_ADD: "false"
|
Co-authored-by: sharknoon <frank.josua@gmail.com>
Thank you for your implementations 😀 If using the proxy causes connection issues, then I also think we do not need to use it. Especially because this app has it's own authentication. Does your camera work now if you are using |
Nice work @joaovictor-local! I just tested and we are almost there. @sharknoon was on the right track with using
^ So for this, the browser isn't making a direct request to NETWORK_IP (or GATEWAY_IP in our case now). Instead it is making a request to something like I've tested with Can you please change the HASS_URL to We'll get the gallery assets prepared in the meantime. |
@joaovictor-local @sharknoon I'm actually wrong about HASS_URL is used for this: If we use Apologies for missing this @joaovictor-local, I see you were getting at something similar here: #1185 (comment) So we're stuck using Error occurred while trying to proxy: umbrel.local:5023/api/camera_proxy/camera.192_168_4_71?token=1baf2552af54b4c921d4ebc29b246122060e3265d12481fe73b4cb73d770c2de&date=1726039586464 Interestingly, I just tried testing this commit 98515b2 @joaovictor-local where you're using the local IP and the camera works using this, BUT unfortunately you have to be accessing HA-Fusion from |
@sharknoon @joaovictor-local I managed to get it working. Can you please give it a test as well? I pushed some changes to finalize the submission, but the main commit with the fix is 798dba6, which adds extra_hosts:
- "${DEVICE_DOMAIN_NAME}:${APP_HAFUSION_MACHINE_IP}" This adds a hostname:ip mapping to the /etc/hosts file in the container and solves our issue. I tested, and this solution works even when accessing HA-Fusion over Tailscale |
@nmfretz I have tested it on my Raspberry Pi and it works perfectly fine :) |
Thanks for testing @sharknoon! And thanks again for this submission @joaovictor-local! Gallery assets are ready and we are going live: |
App Submission
App name
Home Assistant Fusion UI
256x256 SVG icon
Gallery images