TypeError: Object of type date is not JSON serializable : Admin API patch method about pygeoapi

[tomkralidis]

@1ly1mk can you provide a payload that I can use to reproduce? Thanks

[1ly1mk]

@tomkralidis here is the payload for PATCH

{
      "metadata": {
        "identification": {
            "title": "pygeoapi  admin api test",
            "description": "pygeoapi provides an API to geospatial data",
            "keywords": [
                "geospatial",
                "data",
                "api"
            ],
            "keywords_type": "theme",
            "terms_of_service": "https://creativecommons.org/licenses/by/4.0/",
            "url": "http://example.org"
        },
        "license": {
            "name": "CC-BY 4.0 license",
            "url": "https://creativecommons.org/licenses/by/4.0/"
        },
        "provider": {
            "name": "Organization Name",
            "url": "https://pygeoapi.io"
        },
        "contact": {
            "name": "Lastname, Firstname",
            "position": "Position Title",
            "address": "Mailing Address",
            "city": "City",
            "stateorprovince": "Administrative Area",
            "postalcode": "Zip or Postal Code",
            "country": "Country",
            "phone": "+xx-xxx-xxx-xxxx",
            "fax": "+xx-xxx-xxx-xxxx",
            "email": "[email protected]",
            "url": "Contact URL",
            "hours": "Hours of Service",
            "instructions": "During hours of service.  Off on weekends.",
            "role": "pointOfContact"
        }
    }
}

[tomkralidis]

Looks like the existing YAML config that gets parsed into a dict preserves a native datetime value. We could write dates in YAML as literal strings, but that is assuming users are doing same. We could also "round trip" the YAML->dict->JSON->dict as follows:

diff --git a/pygeoapi/config.py b/pygeoapi/config.py
index d1bf5c3..f542325 100644
--- a/pygeoapi/config.py
+++ b/pygeoapi/config.py
@@ -59,7 +59,7 @@ def get_config(raw: bool = False) -> dict:
         else:
             CONFIG = yaml_load(fh)
 
-    return CONFIG
+    return json.loads(to_json(CONFIG))
 
 
 def load_schema() -> dict:

however this seems like extra hops/more moving parts.

cc'ing @webb-ben here for thoughts/input.