Mobrovac has submitted this change and it was merged.

Change subject: Tests: Set the protocol scheme to HTTPS
......................................................................


Tests: Set the protocol scheme to HTTPS

BetaCluster is now enforcing HTTPS and returns a 403 for HTTP requests,
so use HTTPS for every request to the MW and RESTBase APIs.

Change-Id: Iaeb4b86ef591409d40be4350f301a134cecb2388
---
M config.dev.yaml
M lib/api-util.js
2 files changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Mobrovac: Verified; Looks good to me, approved
  jenkins-bot: Verified



diff --git a/config.dev.yaml b/config.dev.yaml
index b3f0858..c0e04e2 100644
--- a/config.dev.yaml
+++ b/config.dev.yaml
@@ -64,14 +64,14 @@
       # the template used for contacting the MW API
       mwapi_req:
         method: post
-        uri: '{{scheme}}://{{domain}}/w/api.php'
+        uri: 'https://{{domain}}/w/api.php'
         headers:
           user-agent: '{{user-agent}}'
         body: '{{ default(request.query, {}) }}'
       # the template used for contacting RESTBase
       restbase_req:
         method: get
-        uri: '{{scheme}}://{{domain}}/api/rest_v1/{+path}'
+        uri: 'https://{{domain}}/api/rest_v1/{+path}'
         query: '{{ default(request.query, {}) }}'
         headers: '{{request.headers}}'
         body: '{{request.body}}'
diff --git a/lib/api-util.js b/lib/api-util.js
index 0f58fdf..06a9ae9 100644
--- a/lib/api-util.js
+++ b/lib/api-util.js
@@ -23,8 +23,7 @@
 
     var request = app.mwapi_tpl.expand({
         request: {
-            params: { domain: domain,
-                      scheme: domain.indexOf('beta.wmflabs.org') !== -1 ? 
'http' : 'https' },
+            params: { domain: domain },
             headers: { 'user-agent': app.conf.user_agent },
             query: query
         }

-- 
To view, visit https://gerrit.wikimedia.org/r/309003
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaeb4b86ef591409d40be4350f301a134cecb2388
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Mobrovac <mobro...@wikimedia.org>
Gerrit-Reviewer: BearND <bsitzm...@wikimedia.org>
Gerrit-Reviewer: Dbrant <dbr...@wikimedia.org>
Gerrit-Reviewer: Fjalapeno <cfl...@wikimedia.org>
Gerrit-Reviewer: GWicke <gwi...@wikimedia.org>
Gerrit-Reviewer: Jhernandez <jhernan...@wikimedia.org>
Gerrit-Reviewer: Mholloway <mhollo...@wikimedia.org>
Gerrit-Reviewer: Mhurd <mh...@wikimedia.org>
Gerrit-Reviewer: Mobrovac <mobro...@wikimedia.org>
Gerrit-Reviewer: Niedzielski <sniedziel...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to