Add bbox validation - Githubissues

mapbox / Hecate

Fast Geospatial Feature Storage API

MIT License

252 stars 36 forks source link

Add bbox validation #178

Closed samely closed 4 years ago

samely commented 4 years ago

Context

This PR fixes https://github.com/mapbox/Hecate/issues/66

There is no validation for bbox parameter.

Summary of changes

[x] Move validation of array length to validate/mod.rs, and call the validate module(?) in feature/mod.rs
[x] Added a bbox validation function to validate/mod.rs, and its unit test.
[x] Added tests to integration tests for bbox validation

Next steps

[x] Review and merge

cc/ @ingalls @lizziegooding @miccolis