Skip to content

Include total length for VODs#168

Closed
mrtncode wants to merge 2 commits intostarkillerOG:mainfrom
mrtncode:patch-1
Closed

Include total length for VODs#168
mrtncode wants to merge 2 commits intostarkillerOG:mainfrom
mrtncode:patch-1

Conversation

@mrtncode
Copy link
Copy Markdown

Required for home-assistant/core#169184

This PR adds total length for VODs. That is required for the core PR because we need the total length/ bytes to get the webkit streaming working.

The code is not perfect. But it works. Feel free to help me and/ or improve the code quality :)

@mrtncode
Copy link
Copy Markdown
Author

@starkillerOG Can you take a look? :)

@starkillerOG
Copy link
Copy Markdown
Owner

Thanks for these efforts!
I am currently on a business trip abroad so I won't have acces to my dev PC for the comming 2 weeks, but when I am back I will take a look.

@mrtncode
Copy link
Copy Markdown
Author

@starkillerOG Thanks for quick update!
I will improve and polish my PRs until you are back :)

@starkillerOG
Copy link
Copy Markdown
Owner

Are you using AI to assist you in writing code?
If so which AI?

@mrtncode
Copy link
Copy Markdown
Author

Are you using AI to assist you in writing code? If so which AI?

I have used Copilot Assistance to to get the file size working. Its definetely not the cleanest code at the moment. I am thankful if you have some tips to improve it/ get file size more directly.
I decided to just do a minimal prototype to get it working and then focus on code quality. The problem is that webkit/ safari is very picky for media streaming and Apple hasnt released clear instructions for developers. Its more try and error :D

@mrtncode
Copy link
Copy Markdown
Author

I will close this for now. It just makes everything too complicated. I think the better solution would be to improve the reolink proxying. In the reolink web ui VOD streaming is not a problem for apple devices.

@mrtncode mrtncode closed this Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants