Merge pull request #385 from aholachek/change-logout

changed logout endpoint from GET to POST
adsabs · May 11, 2015 · 885f154 · 885f154
2 parents 068b4fb + 1c83d4a
commit 885f154
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/js/components/session.js b/src/js/components/session.js
@@ -112,7 +112,7 @@ define([
           query : new ApiQuery({}),
           options : {
             context : this,
-            type : "GET",
+            type : "POST",
             headers : {'X-CSRFToken' :  csrfToken },
             contentType : "application/json",
             done : this.logoutSuccess

diff --git a/test/mocha/js/components/session.spec.js b/test/mocha/js/components/session.spec.js
@@ -69,7 +69,7 @@ define([
 
       expect(requestStub.args[1][0]).to.be.instanceof(ApiRequest)
       expect(requestStub.args[1][0].toJSON().target).to.eql("accounts/logout");
-      expect(requestStub.args[1][0].toJSON().options.type).to.eql("GET");
+      expect(requestStub.args[1][0].toJSON().options.type).to.eql("POST");
       expect(requestStub.args[1][0].toJSON().options.done).to.eql(s.logoutSuccess);
 
       s.register({email: "[email protected]", password1 : "foo", password2 : "foo", "g-recaptcha-response" : "boo"});