Merge pull request #278 from amirmasud/return-204-when-no-tile-found
Return 204 when mbtiles error is 'Tile does not exist'
This commit is contained in:
commit
55fa8bf8c1
2 changed files with 2 additions and 2 deletions
|
@ -89,7 +89,7 @@ module.exports = function(options, repo, params, id, styles) {
|
||||||
source.getTile(z, x, y, function(err, data, headers) {
|
source.getTile(z, x, y, function(err, data, headers) {
|
||||||
if (err) {
|
if (err) {
|
||||||
if (/does not exist/.test(err.message)) {
|
if (/does not exist/.test(err.message)) {
|
||||||
return res.status(404).send(err.message);
|
return res.status(204).send(err.message);
|
||||||
} else {
|
} else {
|
||||||
return res.status(500).send(err.message);
|
return res.status(500).send(err.message);
|
||||||
}
|
}
|
||||||
|
|
|
@ -23,6 +23,6 @@ describe('Vector tiles', function() {
|
||||||
testTile(prefix, 0, 1, 0, 404);
|
testTile(prefix, 0, 1, 0, 404);
|
||||||
testTile(prefix, 0, 0, 1, 404);
|
testTile(prefix, 0, 0, 1, 404);
|
||||||
|
|
||||||
testTile(prefix, 14, 0, 0, 404); // non existent tile
|
testTile(prefix, 14, 0, 0, 204); // non existent tile
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in a new issue