Allow null value in bbox property (#252)

Author: urschrei

CHANGES.md

@@ -14,6 +14,8 @@
   `quick_collection` for conventional naming and simpler docs.
   * <https://github.com/georust/geojson/pulls/214>
 * Added `GeoJson::to_string_pretty` as convenience wrappers around the same `serde_json` methods.
+* The `bbox` property of a `Feature` can now be `null` (returning `None`) when parsing GeoJSON strings in order to facilitate easier processing of GeoJSON "in the wild" (e.g. Copernicus STAC GeoJSON). [The spec](https://datatracker.ietf.org/doc/html/rfc7946#section-5) does not allow this, but implementations appear to disagree.
+  * <https://github.com/georust/geojson/pull/252>
 
 ## 0.24.1
 

src/feature.rs

@@ -307,6 +307,22 @@ mod tests {
         )
     }
 
+    #[test]
+    fn null_bbox() {
+        let geojson_str = r#"{
+            "geometry": null,
+            "bbox": null,
+            "properties":{},
+            "type":"Feature"
+        }"#;
+        let geojson = geojson_str.parse::<GeoJson>().unwrap();
+        let feature = match geojson {
+            GeoJson::Feature(feature) => feature,
+            _ => unimplemented!(),
+        };
+        assert!(feature.bbox.is_none());
+    }
+
     #[test]
     fn test_display_feature() {
         let f = feature().to_string();

src/util.rs

@@ -78,8 +78,8 @@ pub fn get_coords_value(object: &mut JsonObject) -> Result<JsonValue> {
 /// Used by FeatureCollection, Feature, Geometry
 pub fn get_bbox(object: &mut JsonObject) -> Result<Option<Bbox>> {
     let bbox_json = match object.remove("bbox") {
+        Some(JsonValue::Null) | None => return Ok(None),
         Some(b) => b,
-        None => return Ok(None),
     };
     let bbox_array = match bbox_json {
         JsonValue::Array(a) => a,