Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pileup emits HTTP requests to a bigbed with invalid header : Range:bytes=9074145-NaN #446

Open
max-l opened this issue Dec 6, 2016 · 2 comments
Assignees

Comments

@max-l
Copy link

max-l commented Dec 6, 2016

This JSFiddle illustrates the problem :

https://jsfiddle.net/maxl/srx4qecL/4/

In the chrome debug/network console, I see a few successful http requests to the bigbed, and eventually there is a request with the header : Range:bytes=9074145-NaN
the server (nginx) then returns a 406 (Requested Range Not Satisfiable)

something causes pileup to emit a range with NaN ...

The same bigbed file works with other browsers (UCSC, dalliance)

Accept:*/*
Accept-Encoding:gzip, deflate, sdch, br
Accept-Language:en-US,en;q=0.8,fr;q=0.6
Cache-Control:no-cache
Connection:keep-alive
DNT:1
Host:test-roucoulab.ccs.usherbrooke.ca
Origin:https://fiddle.jshell.net
Pragma:no-cache
Range:bytes=9074145-NaN
Referer:https://fiddle.jshell.net/_display/
User-Agent:Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.100 Safari/537.36
@max-l max-l changed the title Pileup emits requests to bigbed with header Range:bytes=9074145-NaN Pileup emits HTTP requests to a bigbed with invalid header : Range:bytes=9074145-NaN Dec 7, 2016
@armish armish self-assigned this Dec 10, 2016
@armish
Copy link
Member

armish commented Dec 14, 2016

thank you so much for reporting this issue, @max-l! I was hoping to get to this soon, but couldn't as other things got in the way. I will be away for about a month, so no promises here but I will try to see whether I can squeeze a debugging/fixing session while I am away.

If this is urgent and has been blocking you, @ihodes might help you with the issue.

Let me know and sorry for the late feedback on this.

@max-l
Copy link
Author

max-l commented Dec 20, 2016

@armish thanks for the quick response, I've updated the JSFiddle to make sure it reproduces the bug

https://jsfiddle.net/srx4qecL/6/

I will be thrilled if @ihodes could look at the problem, but I understand that this is open source and you guys are already kind enough to have developed this awesome project !

One thing that would be awesome is if pileup would use webpack https://webpack.github.io/docs/

it would make and building and contributing to the project easier...

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

No branches or pull requests

2 participants