Merge pull request lloyd#22 from shane-tomlinson/null_assertion_nosignin

Only do the login request if assertion !== null closes lloyd#19
graingert · Sep 23, 2011 · 38d3e55 · 38d3e55
2 parents 766937a + f336669
commit 38d3e55
Showing 1 changed file with 17 additions and 12 deletions.
diff --git a/static/js/main.js b/static/js/main.js
@@ -96,18 +96,23 @@ function loggedOut() {
 // browserid dialog
 function gotVerifiedEmail(assertion) {
   // got an assertion, now send it up to the server for verification
-  $.ajax({
-    type: 'POST',
-    url: '/api/login',
-    data: { assertion: assertion },
-    success: function(res, status, xhr) {
-      if (res === null) loggedOut();
-      else loggedIn(res);
-    },
-    error: function(res, status, xhr) {
-      alert("login failure" + res);
-    }
-  });
+  if (assertion !== null) {
+    $.ajax({
+      type: 'POST',
+      url: '/api/login',
+      data: { assertion: assertion },
+      success: function(res, status, xhr) {
+        if (res === null) loggedOut();
+        else loggedIn(res);
+      },
+      error: function(res, status, xhr) {
+        alert("login failure" + res);
+      }
+    });
+  }
+  else {
+    loggedOut();
+  }
 }
 
 // For some reason, login/logout do not respond when bound using jQuery