Refactor login function and add error handling

src-tauri/src/fortinet.rs

@@ -67,7 +67,7 @@ pub async fn login(
 
     // let session_id = keep_alive_url.split('?').collect::<Vec<&str>>()[1];
     let session_id = keep_alive_url.split("?").nth(1).ok_or("Session id not found")?;
-    println!("{}", session_id);
+    println!("Logged in with sid: {}", session_id);
 
     Ok(())
 }

src-tauri/src/handler.rs

@@ -2,16 +2,19 @@ use std::{fs::File, io::Write};
 
 use tauri::Manager;
 
-use crate::{worker::worker, Credentials};
+use crate::{fortinet, worker::worker, Credentials};
 
 #[tauri::command]
 pub async fn store_credentials(
     app_handle: tauri::AppHandle,
     username: String,
     password: String,
-) -> Result<String, ()> {
+) -> Result<String, String> {
     let s = format!("{}\n{}", username, password);
     println!("{}", s);
+
+    // check credentials valid or not, by logging in
+    fortinet::login(&username, &password).await.or(Err("Login failed"))?;
 
     // write to file
     let app_data_path = app_handle.path_resolver().app_data_dir().unwrap();

src-tauri/src/worker.rs

@@ -14,7 +14,6 @@ pub async fn worker(
         let res = fortinet::login(&username, &password).await;
 
         match res {
-            Ok(_) => println!("Logged in!"),
             Err(e) => {
                 println!("Error: {}", e);
                 if tries == 1 {
@@ -24,6 +23,7 @@ pub async fn worker(
                 sleep(Duration::from_secs(1)).await;
                 continue;
             }
+            Ok(_) => {}
         }
 
         sleep(Duration::from_secs(SLEEP_TIME)).await;

src/main.js

@@ -6,7 +6,11 @@ let credsMsgEl;
 
 async function store_credentials() {
   console.log("creds", credsUsernameEl.value, credsPasswordEl.value)
-  credsMsgEl.textContent = await invoke("store_credentials", { username: credsUsernameEl.value, password: credsPasswordEl.value });
+  try{
+    credsMsgEl.textContent = await invoke("store_credentials", { username: credsUsernameEl.value, password: credsPasswordEl.value });
+  }catch(e){
+    credsMsgEl.textContent = e;
+  }
 }
 
 window.addEventListener("DOMContentLoaded", () => {