Merge pull request #128 from amplitude/force-https-true

Default forceHtttps to true
amplitude · Nov 21, 2017 · 74ba919 · 74ba919
2 parents 6e12116 + 1a33cf1
commit 74ba919
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/src/options.js b/src/options.js
@@ -21,7 +21,7 @@ export default {
   batchEvents: false,
   eventUploadThreshold: 30,
   eventUploadPeriodMillis: 30 * 1000, // 30s
-  forceHttps: false,
+  forceHttps: true,
   includeGclid: false,
   saveParamsReferrerOncePerSession: true,
   deviceIdFromUrlParam: false,

diff --git a/test/amplitude-client.js b/test/amplitude-client.js
@@ -1069,6 +1069,7 @@ describe('setVersionName', function() {
     });
 
     it('should send request', function() {
+      amplitude.options.forceHttps = false;
       amplitude.logEvent('Event Type 1');
       assert.lengthOf(server.requests, 1);
       assert.equal(server.requests[0].url, 'http://api.amplitude.com/');

diff --git a/test/amplitude.js b/test/amplitude.js
@@ -1057,6 +1057,7 @@ describe('setVersionName', function() {
     });
 
     it('should send request', function() {
+      amplitude.options.forceHttps = false;
       amplitude.logEvent('Event Type 1');
       assert.lengthOf(server.requests, 1);
       assert.equal(server.requests[0].url, 'http://api.amplitude.com/');