ui: avoid "async race" with new client POST

The getClients() and POST request were being issued one after another (not waiting for the POST to return). This can cause the GET /clients to return the old list before the user config is updated. Waiting for the POST to return before retrieving the new client list fixes the race. This behavior can be reproduced by adding a time.Sleep(time.Second) to the CreateClient server handler.
EmbarkStudios · Jan 17, 2020 · e176ae0 · e176ae0
1 parent 8fc4378
commit e176ae0
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/ui/src/Clients.svelte b/ui/src/Clients.svelte
@@ -17,9 +17,10 @@
   async function handleNewClick(event) {
     const res = await fetch(clientsUrl, {
       method: "POST",
-    }).then(getClients());
+    });
     let newClient = await res.json();
     console.log("New client added", newClient);
+    await getClients();
   }
 
 	onMount(getClients);