Uncomment Oauth (#1113)

* uncomment oauth

* Remove Failed to authenticate ... message in case of successful authentication

Author: cx-elchanan-arbiv

media/auth.js

@@ -86,6 +86,7 @@
           loginForm.classList.add("hidden");
           authenticatedMessage.classList.remove("hidden");
           logoutButton.classList.remove("hidden");
+          messageBox.style.display = "none";
         } else {
           loginForm.classList.remove("hidden");
           authenticatedMessage.classList.add("hidden");
@@ -121,16 +122,16 @@
       } else if (message.type === "clearFields") {
         document.querySelector(
           'input[name="authMethod"][value="oauth"]'
-        ).checked = false;
+        ).checked = true;
         document.querySelector(
           'input[name="authMethod"][value="apiKey"]'
-        ).checked = true;
+        ).checked = false;
         apiKeyInput.value = "";
         urlInput.value = "";
         tenantInput.value = "";
         authButton.disabled = true;
         document
-          .querySelector('input[name="authMethod"][value="apikey"]')
+          .querySelector('input[name="authMethod"][value="oauth"]')
           .dispatchEvent(new Event("change"));
       }
     });

src/webview/authenticationWebview.ts

@@ -145,16 +145,16 @@ export class AuthenticationWebview {
         <div id="loginForm">
         <div class="radio-group">
         
-            <label style="display: none;">
+            <label>
                 <input type="radio" name="authMethod" value="oauth" checked> OAuth
             </label>
       
             <label>
-                <input type="radio" name="authMethod" value="apiKey" checked>API Key
+                <input type="radio" name="authMethod" value="apiKey">API Key
             </label>
         </div>
 
-        <div style="display: none;" id="oauthForm" class="auth-form">
+        <div  id="oauthForm" class="auth-form">
             <label for="baseUri" class="form-label">Checkmarx One Base URL:</label>
             <input type="text" id="baseUri" class="auth-input" placeholder="Enter Checkmarx One Base URL">
             <div id="urls-list" class="autocomplete-items"></div>
@@ -167,7 +167,7 @@ export class AuthenticationWebview {
         </div>
 
              <!-- (We need to return it to the next div ) (class="hidden">)   -->
-        <div id="apiKeyForm" 
+        <div id="apiKeyForm" class="hidden"> 
           <label for="apiKey" class="form-label">Checkmarx One API Key:</label>
 			    <input type="password" id="apiKey" placeholder="Enter Checkmarx One API Key" class="auth-input">
         </div>