fix: handle 303 status code

graasp · Apr 12, 2022 · c75b108 · c75b108
1 parent bcd10da
commit c75b108
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 4 deletions.
diff --git a/src/api/authentication.js b/src/api/authentication.js
@@ -1,5 +1,10 @@
 import { API_HOST } from '../config/constants';
-import { DEFAULT_GET, DEFAULT_POST, checkRequest } from './utils';
+import {
+  DEFAULT_GET,
+  DEFAULT_POST,
+  checkRequest,
+  checkSeeOther,
+} from './utils';
 
 // payload = {email}
 export const signIn = async (payload) => {
@@ -15,7 +20,7 @@ export const signInPassword = async (payload) => {
   const req = await fetch(`${API_HOST}/loginpassword`, {
     ...DEFAULT_POST,
     body: JSON.stringify(payload),
-  }).then(checkRequest);
+  }).then(checkSeeOther);
   const data = await req.json();
   return data.link;
 };

diff --git a/src/api/utils.js b/src/api/utils.js
@@ -26,10 +26,10 @@ export const DEFAULT_PUT = {
   credentials: 'include',
 };
 
-export class CustomError extends Error { 
+export class CustomError extends Error {
   response;
 
-  constructor(message, response){
+  constructor(message, response) {
     super(message);
     this.response = response;
   }
@@ -43,3 +43,11 @@ export const checkRequest = (res) => {
 
   throw new CustomError(res.statusText, res);
 };
+export const checkSeeOther = (res) => {
+  if (res.status === 303) {
+    // res.status >= 200 && res.status < 300
+    return res;
+  }
+
+  throw new CustomError(res.statusText, res);
+};