This is an automated email from the ASF dual-hosted git repository.
skperez pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-sdap-nexus.git
The following commit(s) were added to refs/heads/master by this push:
new 2b056aa SDAP-449: Fix for NetCDF handling in SUI (#236)
2b056aa is described below
commit 2b056aa68fc74074d4760266b586c4c4840641dc
Author: Riley Kuttruff <[email protected]>
AuthorDate: Fri Mar 17 11:48:26 2023 -0700
SDAP-449: Fix for NetCDF handling in SUI (#236)
Co-authored-by: rileykk <[email protected]>
---
CHANGELOG.md | 2 ++
analysis/webservice/apidocs/dataset-populate.js | 2 +-
analysis/webservice/apidocs/openapi.yml | 4 ++--
.../webservice/nexus_tornado/request/renderers/NexusNETCDFRenderer.py | 2 +-
4 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8fc40e1..0aa4b91 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -23,6 +23,8 @@ and this project adheres to [Semantic
Versioning](https://semver.org/spec/v2.0.0
- Made `platforms` param optional in `/cdmssubset`, and removed int requirement
- Updated OpenAPI specification for `/cdmssubset` to accurately reflect
`platforms` and `parameter` field options.
- SDAP-436: Added special case for handling Cassandra SwathMulti tiles with
uniform time arrays
+- SDAP-449: Fixed `/cdmsresults` NetCDF output displaying and downloading as
.txt.
+- SDAP-449: Fixed 404 error when populating datasets; script was still using
`/domslist`
### Security
## [1.0.0] - 2022-12-05
diff --git a/analysis/webservice/apidocs/dataset-populate.js
b/analysis/webservice/apidocs/dataset-populate.js
index 6e447eb..89c3f82 100644
--- a/analysis/webservice/apidocs/dataset-populate.js
+++ b/analysis/webservice/apidocs/dataset-populate.js
@@ -15,7 +15,7 @@
function DatasetPopulatePlugin(system) {
- const DS_PATH = '/domslist'
+ const DS_PATH = '/cdmslist'
const ACTION_UPDATE_DATASETS = 'UPDATE_DATASETS'
const ACTION_UPDATE_ERROR = 'UPDATE_ERROR'
diff --git a/analysis/webservice/apidocs/openapi.yml
b/analysis/webservice/apidocs/openapi.yml
index 36982af..55802cd 100644
--- a/analysis/webservice/apidocs/openapi.yml
+++ b/analysis/webservice/apidocs/openapi.yml
@@ -500,7 +500,7 @@ paths:
get:
summary: |
Convert matchup result
- operationId: domsresults
+ operationId: cdmsresults
tags:
- Matchup
parameters:
@@ -529,7 +529,7 @@ paths:
text/csv:
schema:
type: string
- application/octet-stream:
+ application/x-netcdf:
schema:
type: string
format: binary
diff --git
a/analysis/webservice/nexus_tornado/request/renderers/NexusNETCDFRenderer.py
b/analysis/webservice/nexus_tornado/request/renderers/NexusNETCDFRenderer.py
index 00970de..9049d26 100644
--- a/analysis/webservice/nexus_tornado/request/renderers/NexusNETCDFRenderer.py
+++ b/analysis/webservice/nexus_tornado/request/renderers/NexusNETCDFRenderer.py
@@ -24,7 +24,7 @@ class NexusNETCDFRenderer(object):
def render(self, tornado_handler, result):
tornado_handler.set_header("Content-Type", "application/x-netcdf")
- tornado_handler.set_header("Content-Disposition", "filename=\"%s\"" %
self._request.get_argument('filename', "download.nc"))
+ tornado_handler.set_header("Content-Disposition", "attachment;
filename=\"%s\"" % self._request.get_argument('filename', "download.nc"))
try:
tornado_handler.write(result.toNetCDF())
except: