Skip to content

Update README.md#87

Merged
droans merged 1 commit intodroans:mainfrom
shalak:patch-1
Jan 4, 2026
Merged

Update README.md#87
droans merged 1 commit intodroans:mainfrom
shalak:patch-1

Conversation

@shalak
Copy link
Copy Markdown
Contributor

@shalak shalak commented Jan 4, 2026

I believe this section should be moved up to the corresponding topic.

Also - correct me if I'm wrong, but if my Jellyfin instance is HTTPS (as it Home Assistant) - I won't have this issue, right?

I believe this section should be moved up to the corresponding topic.

Also - correct me if I'm wrong, but if my Jellyfin instance is HTTPS (as it Home Assistant) - I won't have this issue, right?
@droans droans added the documentation Solely about the documentation of the project. label Jan 4, 2026
@droans
Copy link
Copy Markdown
Owner

droans commented Jan 4, 2026

Thanks - merged!

Also - correct me if I'm wrong, but if my Jellyfin instance is HTTPS (as it Home Assistant) - I won't have this issue, right?

It's... complicated.

  • MA might connect via mDNS which would use an HTTP connection
  • You would likely need both HTTPS and a domain - eg, just connecting to the IP won't be enough
  • Jellyfin might report back a local address anyways.

But beyond all that, my code for grabbing the image is rather poor and the schema is just as bad. I need to base the approach on the media type given. The response will also need to include all fields - type, path, provider, and remotely_accessible

@droans droans merged commit e6e8ab8 into droans:main Jan 4, 2026
4 of 5 checks passed
@shalak shalak deleted the patch-1 branch January 4, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Solely about the documentation of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants