Make sure Bond is called via HTTPS (#149)

* WA-155 Bond is forcing HTTPS and superagent doesn't seem to handle/follow the redirect response right when calling via HTTP. Request should be HTTPS anyway. * WA-155 Added logging to help debug http requests to bond and DRS resolver.
broadinstitute · Apr 10, 2020 · eed7703 · eed7703
1 parent 7087b04
commit eed7703
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/common/api_adapter.js b/common/api_adapter.js
@@ -15,6 +15,7 @@ function get(method, url, authorization) {
         req.set('authorization', authorization);
     }
 
+    console.log('Making Request: ' + JSON.stringify(req));
     return req;
 }
 

diff --git a/config.json b/config.json
@@ -3,7 +3,7 @@
 
 
 {
-    "dataObjectResolutionHost": "dataguids.org",
+    "dataObjectResolutionHost": "staging.gen3.biodatacatalyst.nhlbi.nih.gov",
     "bondBaseUrl": "broad-bond-dev.appspot.com",
     "samBaseUrl": "https://sam.dsde-dev.broadinstitute.org"
 }
diff --git a/config.json.ctmpl b/config.json.ctmpl
@@ -4,7 +4,7 @@
 
 {
     "dataObjectResolutionHost": {{if eq $runContext "fiab"}}"wb-mock-drs-dev.storage.googleapis.com"{{else if eq $environment "prod"}}"gen3.biodatacatalyst.nhlbi.nih.gov"{{else}}"staging.gen3.biodatacatalyst.nhlbi.nih.gov"{{end}},
-    "bondBaseUrl": {{if eq $runContext "fiab"}}"bond-fiab.{{$dnsDomain}}"{{else}}"broad-bond-{{$environment}}.appspot.com"{{end}},
+    "bondBaseUrl": {{if eq $runContext "fiab"}}"bond-fiab.{{$dnsDomain}}"{{else}}"https://broad-bond-{{$environment}}.appspot.com"{{end}},
     "samBaseUrl": {{if eq $runContext "fiab"}}"https://sam-fiab.{{$dnsDomain}}"{{else}}"https://sam.dsde-{{$environment}}.broadinstitute.org"{{end}}
 }
 {{end}}{{end}}{{end}}
-Original file line number
+Diff line change
@@ Expand Up / @@ -15,6 +15,7 @@ function get(method, url, authorization) { @@
             req.set('authorization', authorization);
         }
+        console.log('Making Request: ' + JSON.stringify(req));
         return req;
     }
@@ Expand Down @@