Add HTTP method parameter for fetch calls.

nathan-osman · Feb 19, 2023 · e075c66 · e075c66
1 parent c05fa06
commit e075c66
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 15 deletions.
diff --git a/ui/src/components/header.js b/ui/src/components/header.js
@@ -26,7 +26,7 @@ export default function Header() {
 
   function handleLogout(e) {
     e.preventDefault()
-    api.fetch('/auth/logout', {})
+    api.fetch('/auth/logout', 'post')
       .then(() => {
         navigate('/login')
       })

diff --git a/ui/src/components/tool.js b/ui/src/components/tool.js
@@ -13,7 +13,7 @@ export default function Tool({ src, url, data, ...props }) {
 
   function handleClick() {
     setIsLoading(true)
-    api.fetch(url, data)
+    api.fetch(url, method, data)
       .catch(e => popup.error(e.message))
       .finally(() => setIsLoading(false))
   }

diff --git a/ui/src/lib/api.js b/ui/src/lib/api.js
@@ -14,16 +14,11 @@ function ApiProvider({ children }) {
   const [isActive, setIsActive] = useState(false)
   const [status, setStatus] = useState(null)
 
-  async function fetchInternal(url, data) {
-    let init
-    if (typeof data !== 'undefined') {
-      init = {
-        method: 'POST',
-        headers: {
-          'Content-Type': 'application/json'
-        },
-        body: JSON.stringify(data)
-      }
+  async function fetchInternal(url, method, data) {
+    let init = { method }
+    if (data != undefined) {
+      init['headers'] = { 'Content-Type': 'application/json' }
+      init['body'] = JSON.stringify(data)
     }
     const response = await fetch(url, init)
     if (!response.ok) {
@@ -41,10 +36,10 @@ function ApiProvider({ children }) {
   const api = {
     isActive,
     status,
-    fetch: async (url, data) => {
+    fetch: async (url, method, data) => {
       setIsActive(true)
       try {
-        return await fetchInternal(url, data)
+        return await fetchInternal(url, method, data)
       } finally {
         setIsActive(false)
       }

diff --git a/ui/src/routes/login.js b/ui/src/routes/login.js
@@ -19,7 +19,7 @@ export default function Login() {
 
   function handleSubmit(e) {
     e.preventDefault()
-    api.fetch('/auth/login', { username, password })
+    api.fetch('/auth/login', 'post', { username, password })
       .then(() => {
         navigate(url)
       })