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

Send 404 error in case of compressed file not exists #3508

Closed
SunBlack opened this issue Feb 2, 2016 · 7 comments
Closed

Send 404 error in case of compressed file not exists #3508

SunBlack opened this issue Feb 2, 2016 · 7 comments

Comments

@SunBlack
Copy link

SunBlack commented Feb 2, 2016

Since revision 90fd9f5 you don't get anymore a 404 error in case your browser want to load a tile which not exists. Instead you get always a ERR_CONTENT_DECODING_FAILED (same message occurs in case tile data are not gzip compressed, so you don't see anymore if a tile couldn't be loaded because of tile data are missing or data are not compressed).

I could preorduce this issue with all gzip compressed files (b3dm, pnts, i3dm and cmpt).

@pjcozzi
Copy link
Contributor

pjcozzi commented Feb 2, 2016

Thanks we'll look at this as part of #3241.

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 13, 2016

@SunBlack does #3839 work for you?

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 14, 2016

@SunBlack let me know if you run into any issues.

@pjcozzi pjcozzi closed this as completed Apr 14, 2016
@SunBlack
Copy link
Author

Well I had now time to test ist. In my opinion removing gzip compression isn't best solution - or isn't there another solution in case file not exists?

@pjcozzi
Copy link
Contributor

pjcozzi commented May 18, 2016

Tiles can still be served pre-gzipped, we just don't pre-gzip the test data so it is easy for developers to work with.

@SunBlack
Copy link
Author

My tiles are currently pre-zipped, but because of missing http header not extracted by browser. So maybe an it is possible to add an option to default server implementation, like --port, to enable compression support?

@pjcozzi
Copy link
Contributor

pjcozzi commented May 19, 2016

That is a good idea that we can consider. In the meantime, you want to not pre-gzip or add a header like:

res.header('Content-Encoding', 'gzip');

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