wearespindle / django-ranged-fileresponse Goto Github PK

Version

0.1.2

File / Feature

class RangedFileReader(object):
    def __init(...)

Expected behaviour

Obtaining the size of the file should be much quicker, and less CPU intensive. We should probably use something like this:

os.fstat(self.f)
self.size = statobj.st_size

Actual behaviour

The current approach for obtaining the size of the file is terribly slow on large files : it uses CPU and disk I/O
self.size = len(self.f.read())

I suspect this already works with Django 2.0, but we could add it to the test environments.

There's a 0.2.0 version on pypi, but this repo seems stuck at 0.1.2.

I use Safari, and I've been trying to add HTML 5 Video to my Django project for a while, and nothing has worked.

Use Django-Ranged-Response solved the problem. So Thank you.

My only concern, is that I see an open issue that implies this code hasn't been tested against Django 2? And Django is up to 3.07 / 3.1, at this time.

Now the good news, is that I ran into no obvious issues with using it with Django 3.0x. But since Django out of the box doesn't support Ranges, I'd like to know if there is some development going on here? What's the roadmap?

The package is great!!

Pypi is convenient for deployment because install_requires in setup.py only supports pipy but not git.
Would you want to upload it to pipy?

Version

{version or tag here}

File / Feature

{file or feature containing the issue}

Expected behaviour

{what should happen}

Actual behaviour

{what happens}

Stacktrace / Error message

{paste here}

Other info

{anything else that might be related/useful}