add login type

Author: DarylThayil

JavaScriptSPA/index.html

@@ -22,7 +22,8 @@
         clientID: "245e9392-c666-4d51-8f8a-bfd9e55b2456",
             authority: "https://login.microsoftonline.com/common",
             graphScopes: ["user.read", "Mail.Send"],
-            graphEndpoint: "https://graph.microsoft.com/v1.0/me"
+            graphEndpoint: "https://graph.microsoft.com/v1.0/me",
+            loginType: 'POPUP'
     };
 
     var config = {
@@ -140,23 +141,15 @@
             errorMessage.indexOf("interaction_required") !== -1 ||
             errorMessage.indexOf("login_required") !== -1;
     }
-    // Browser check variables
-    var ua = window.navigator.userAgent;
-    var msie = ua.indexOf('MSIE ');
-    var msie11 = ua.indexOf('Trident/');
-    var msedge = ua.indexOf('Edge/');
-    var isIE = msie > 0 || msie11 > 0;
-    var isEdge = msedge > 0;
-
-    //If you support IE, our recommendation is that you sign-in using Redirect APIs
-    //If you as a developer are testing using Edge InPrivate mode, please add "isEdge" to the if check
-    if (!isIE) {
+
+    // runs on page load, change config to try different login types to see what is best for your application
+    if (applicationConfig.loginType === 'POPUP') {
         if (myMSALObj.getAccount()) {// avoid duplicate code execution on page load in case of iframe and popup window.
             showWelcomeMessage();
             acquireTokenPopupAndCallMSGraph();
         }
     }
-    else {
+    else if (applicationConfig.loginType === 'REDIRECT') {
         document.getElementById("SignIn").onclick = function () {
             let redirectTokenRequest = {
                 scopes: applicationConfig.graphScopes
@@ -168,6 +161,8 @@
             showWelcomeMessage();
             acquireTokenRedirectAndCallMSGraph();
         }
+    } else {
+        console.error('Please choose a valid login type');
     }
 </script>
 </body>