You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sdap.apache.org by rk...@apache.org on 2023/10/19 17:09:19 UTC

[incubator-sdap-nexus] branch master updated: Ensure job status links are https (#283)

This is an automated email from the ASF dual-hosted git repository.

rkk 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 9906869  Ensure job status links are https (#283)
9906869 is described below

commit 99068694407a964a1ec5ebd947915887ce0b3ef1
Author: Riley Kuttruff <72...@users.noreply.github.com>
AuthorDate: Thu Oct 19 10:09:14 2023 -0700

    Ensure job status links are https (#283)
    
    * Ensure job status links are https
    
    * Update CHANGELOG.md
---
 CHANGELOG.md                                           | 1 +
 analysis/webservice/algorithms/doms/ExecutionStatus.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 1178918..e848427 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -33,6 +33,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - SDAP-479: Fixed `/cdmssubset` failure for variables without specified standard_name. 
 - SDAP-39: When querying for tiles by polygon, use the poly's bounding box with the bbox methods instead of using Solr's polygon search
 - Status code for results endpoint if execution id is not found fixed to be `404` instead of `500`.
+- Ensured links in the `/job` endpoint are https
 ### Security
 
 ## [1.1.0] - 2023-04-26
diff --git a/analysis/webservice/algorithms/doms/ExecutionStatus.py b/analysis/webservice/algorithms/doms/ExecutionStatus.py
index 1bae455..d21ab44 100644
--- a/analysis/webservice/algorithms/doms/ExecutionStatus.py
+++ b/analysis/webservice/algorithms/doms/ExecutionStatus.py
@@ -54,7 +54,7 @@ class ExecutionStatusHandler(BaseDomsHandler.BaseDomsQueryCalcHandler):
                 )
 
         job_status = NexusExecutionResults.ExecutionStatus(execution_details['status'])
-        host = f'{request.requestHandler.request.protocol}://{request.requestHandler.request.host}'
+        host = f'https://{request.requestHandler.request.host}'
 
         return NexusExecutionResults.NexusExecutionResults(
             status=job_status,