home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.17k stars 30.57k forks source link

Media proxied incorrectly #124849

Open rgcosma opened 2 months ago

rgcosma commented 2 months ago

The problem

HA running behing nginx+letsencrypt, external_url set, I can access it at https://external.url, all good. HA detects a DLNA server on the local network, loads it media library, I can see file listings and properties.

Problem: when clicking "play" on any video file, the URL generated is internal, of the form http://192.168.0.1:8080/file.mp4 Proposed solution: HA should do some form of URL rewriting and proxy the request so that the file is available at https://external.url/file.mp4

What version of Home Assistant Core has the issue?

core-2024.8.3

What was the last working version of Home Assistant Core?

n/a

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

DLNA

Link to integration documentation on our website

https://www.home-assistant.io/integrations/dlna_dms/

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

home-assistant[bot] commented 2 months ago

Hey there @chishm, mind taking a look at this issue as it has been labeled with an integration (dlna_dms) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `dlna_dms` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign dlna_dms` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


dlna_dms documentation dlna_dms source (message by IssueLinks)