skorper opened a new pull request, #197:
URL: https://github.com/apache/incubator-sdap-nexus/pull/197

   * Fixed bug introduced in the last PR where `domsresults` no longer worked. 
This was because the `details` field name were updated, but the code that 
inserts results into the database was not. I've updated the code that inserts 
the details into the database to look for the new field names. `None` is being 
used for `stats["numGriddedChecked"]` and `stats["numInsituChecked"]` because 
otherwise we'd need to change the db schema which would cause issues with many 
existing SDAP deployments. 
   * Added link to CDMS Schema endpoint in Swagger in the `parameter` and 
`platforms` field descriptions
   * Added env var `REQUESTS_CA_BUNDLE` env var to Dockerfile. Building the 
docker image suddenly stopped working the other day with error `Could not find 
a suitable TLS CA certificate bundle`, so this is a fix for that. Both myself 
and @kevinmarlis  have tested this fix.
   
   These changes are deployed to the CDMS AWS `nexus-sit` environment. 
   
   I tested the `domsresults` endpoint with both CDMS and DOMS matchup requests 
and was able to retrieve CSV results for both.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@sdap.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to