fixed login bug

Author: Vivek Chowdhury

src/api/LoginServices.js

@@ -6,7 +6,7 @@ export function validateUser(userid, password) {
 }
 
 async function handleResponse(response) {
-  if (response.ok) return response.json();
+  if (response.ok || response.status === 401) return response.json();
   if (response.status === 400) {
     // So, a server-side validation error occurred.
     // Server side validation returns a string error message, so parse as text instead of json.

src/app/Login/Login.js

@@ -6,6 +6,7 @@ import * as LoginServices from "./../../api/LoginServices";
 
 function Login(props) {
   const [user, setUser] = useState({ userId: "", password: "" });
+  const [error, setError] = useState("");
 
   const handleChange = ({ target }) => {
     setUser({
@@ -19,7 +20,12 @@ function Login(props) {
     console.log(user);
     LoginServices.validateUser(user.userId, user.password).then((response) => {
       console.log(response);
-      props.history.push("/feeds");
+      if (!response.error) {
+        props.history.push("/feeds");
+      } else {
+        setError("Invalid User naem or Password, please try again !");
+        // TODO: Need to display error message
+      }
     });
   };
 
@@ -34,6 +40,7 @@ function Login(props) {
             onTextChange={handleChange}
             {...user}
             onSubmit={handleSubmit}
+            error={error}
           />
         </div>
       </div>

src/app/Login/LoginForm/LoginForm.js

@@ -5,6 +5,7 @@ import { Link } from "react-router-dom";
 function LoginForm(props) {
   return (
     <form className="form-horizontal" onSubmit={props.onSubmit}>
+      {props.error && <div className="alert alert-danger">{props.error}</div>}
       <FormInput
         inputId="userId"
         inputType="text"

src/app/TechMarketShell.js

@@ -26,7 +26,7 @@ function TechMarketShell(props) {
           <Route path="/" exact component={Login} />
           <Route path="/feeds" component={Feeds} />
           {/* <Route path="/profile" component={Profile} /> */}
-          {enforceAuthentication(Profile, "/profile", null)}
+          {enforceAuthentication(Profile, "/profile", {})}
           <Route path="/about" component={About} />
           <Route path="/contact" component={ContactUs} />
         </Switch>

tools/localServer.js

@@ -59,8 +59,8 @@ server.get("/login/", function (req, res, next) {
     }
   }
   if (!match || match.length === 0) {
-    const error = { errorCode: 404, error: "No record found" };
-    res.status(404).send(error);
+    const error = { errorCode: 401, error: "No record found" };
+    res.status(401).send(error);
   }
   // next();
 });
@@ -73,8 +73,8 @@ server.get("/login/validateUserId/", function (req, res, next) {
     const message = { message: "No match found" };
     res.status(200).send(message);
   } else {
-    const error = { errorCode: 404, error: "User id already exist" };
-    res.status(404).send(error);
+    const error = { errorCode: 401, error: "User id already exist" };
+    res.status(401).send(error);
   }
 });