Reply To: The case for reviving Firefly

#18727
Anonymous
Inactive

Julien, I have a patch for you which was needed in order to serve video to my Front Row clients. I tried sending it to you at blache.org, but got the following in return. I also tried sending it to your debian address. If you don’t get it, and do read this, please let me know how to best get it to you.


From: [email protected]
Subject: failure notice
Date: Monday, December 7, 2009, 9:31 PM

Sorry, we were unable to deliver your message to the following address.

:
No MX or A records for jblache.org

Specifically…
* Accepts “QuickTime*” and “Front%20Row*” as user agents
* Returns a Content-Length header for streamed files
* Respects the ending number in a “Range” header
* Returns a correct “Content-Type” for video files

Basically, Front Row is much pickier about how the response headers are constructed than really anything else.

Also, while I was at it, I cleaned up a few things which you may or may not find interesting…

* Streaming file is never closed, as far as I could tell, so I added a close() to stream_end()
* Content-Range was returning one too many bytes on the size.
* Demoted a couple very verbose logging messages to SPAM, which helped a lot using the log files for debugging.