diff --git a/lecture-pulse/package-lock.json b/lecture-pulse/package-lock.json index a5fb354..0465de5 100644 --- a/lecture-pulse/package-lock.json +++ b/lecture-pulse/package-lock.json @@ -10,6 +10,7 @@ "dependencies": { "@radix-ui/react-slot": "^1.2.4", "@tailwindcss/vite": "^4.1.18", + "bcryptjs": "^3.0.3", "class-variance-authority": "^0.7.1", "clsx": "^2.1.1", "framer-motion": "^12.23.26", @@ -71,7 +72,6 @@ "integrity": "sha512-e7jT4DxYvIDLk1ZHmU/m/mB19rex9sv0c2ftBtjSBv+kVM/902eh0fINUzD7UwLLNR+jU585GxUJ8/EBfAM5fw==", "dev": true, "license": "MIT", - "peer": true, "dependencies": { "@babel/code-frame": "^7.27.1", "@babel/generator": "^7.28.5", @@ -1818,7 +1818,6 @@ "integrity": "sha512-MWtvHrGZLFttgeEj28VXHxpmwYbor/ATPYbBfSFZEIRK0ecCFLl2Qo55z52Hss+UV9CRN7trSeq1zbgx7YDWWg==", "devOptional": true, "license": "MIT", - "peer": true, "dependencies": { "csstype": "^3.2.2" } @@ -1873,7 +1872,6 @@ "integrity": "sha512-NZyJarBfL7nWwIq+FDL6Zp/yHEhePMNnnJ0y3qfieCrmNvYct8uvtiV41UvlSe6apAfk0fY1FbWx+NwfmpvtTg==", "dev": true, "license": "MIT", - "peer": true, "bin": { "acorn": "bin/acorn" }, @@ -2121,6 +2119,15 @@ "baseline-browser-mapping": "dist/cli.js" } }, + "node_modules/bcryptjs": { + "version": "3.0.3", + "resolved": "https://registry.npmjs.org/bcryptjs/-/bcryptjs-3.0.3.tgz", + "integrity": "sha512-GlF5wPWnSa/X5LKM1o0wz0suXIINz1iHRLvTS+sLyi7XPbe5ycmYI3DlZqVGZZtDgl4DmasFg7gOB3JYbphV5g==", + "license": "BSD-3-Clause", + "bin": { + "bcrypt": "bin/bcrypt" + } + }, "node_modules/brace-expansion": { "version": "1.1.15", "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-1.1.15.tgz", @@ -2152,7 +2159,6 @@ } ], "license": "MIT", - "peer": true, "dependencies": { "baseline-browser-mapping": "^2.9.0", "caniuse-lite": "^1.0.30001759", @@ -2983,7 +2989,6 @@ "integrity": "sha512-LEyamqS7W5HB3ujJyvi0HQK/dtVINZvd5mAAp9eT5S/ujByGjiZLCzPcHVzuXbpJDJF/cxwHlfceVUDZ2lnSTw==", "dev": true, "license": "MIT", - "peer": true, "dependencies": { "@eslint-community/eslint-utils": "^4.8.0", "@eslint-community/regexpp": "^4.12.1", @@ -4240,7 +4245,6 @@ "resolved": "https://registry.npmjs.org/jspdf/-/jspdf-4.2.1.tgz", "integrity": "sha512-YyAXyvnmjTbR4bHQRLzex3CuINCDlQnBqoSYyjJwTP2x9jDLuKDzy7aKUl0hgx3uhcl7xzg32agn5vlie6HIlQ==", "license": "MIT", - "peer": true, "dependencies": { "@babel/runtime": "^7.28.6", "fast-png": "^6.2.0", @@ -4950,7 +4954,6 @@ "resolved": "https://registry.npmjs.org/picomatch/-/picomatch-4.0.4.tgz", "integrity": "sha512-QP88BAKvMam/3NxH6vj2o21R6MjxZUAd6nlwAS/pnGvN9IVLocLHxGYIzFhg6fUQ+5th6P4dv4eW9jX3DSIj7A==", "license": "MIT", - "peer": true, "engines": { "node": ">=12" }, @@ -5059,7 +5062,6 @@ "resolved": "https://registry.npmjs.org/react/-/react-19.2.3.tgz", "integrity": "sha512-Ku/hhYbVjOQnXDZFv2+RibmLFGwFdeeKHFcOTlrt7xplBnya5OGn/hIRDsqDiSUcfORsDC7MPxwork8jBwsIWA==", "license": "MIT", - "peer": true, "engines": { "node": ">=0.10.0" } @@ -5069,7 +5071,6 @@ "resolved": "https://registry.npmjs.org/react-dom/-/react-dom-19.2.3.tgz", "integrity": "sha512-yELu4WmLPw5Mr/lmeEpox5rw3RETacE++JgHqQzd2dg+YbJuat3jH4ingc+WPZhxaoFzdv9y33G+F7Nl5O0GBg==", "license": "MIT", - "peer": true, "dependencies": { "scheduler": "^0.27.0" }, @@ -5089,7 +5090,6 @@ "resolved": "https://registry.npmjs.org/react-redux/-/react-redux-9.2.0.tgz", "integrity": "sha512-ROY9fvHhwOD9ySfrF0wmvu//bKCQ6AeZZq1nJNtbDC+kk5DuSuNX/n6YWYF/SYy7bSba4D4FSz8DJeKY/S/r+g==", "license": "MIT", - "peer": true, "dependencies": { "@types/use-sync-external-store": "^0.0.6", "use-sync-external-store": "^1.4.0" @@ -5190,8 +5190,7 @@ "version": "5.0.1", "resolved": "https://registry.npmjs.org/redux/-/redux-5.0.1.tgz", "integrity": "sha512-M9/ELqF6fy8FwmkpnF0S3YKOqMyoWJ4+CS5Efg2ct3oY9daQvd/Pc71FpGZsVsbl3Cpb+IIcjBDUnnyBdQbq4w==", - "license": "MIT", - "peer": true + "license": "MIT" }, "node_modules/redux-thunk": { "version": "3.1.0", @@ -6052,7 +6051,6 @@ "resolved": "https://registry.npmjs.org/vite/-/vite-7.3.3.tgz", "integrity": "sha512-/4XH147Ui7OGTjg3HbdWe5arnZQSbfuRzdr9Ec7TQi5I7R+ir0Rlc9GIvD4v0XZurELqA035KVXJXpR61xhiTA==", "license": "MIT", - "peer": true, "dependencies": { "esbuild": "^0.27.0", "fdir": "^6.5.0", @@ -6292,7 +6290,6 @@ "integrity": "sha512-0wZ1IRqGGhMP76gLqz8EyfBXKk0J2qo2+H3fi4mcUP/KtTocoX08nmIAHl1Z2kJIZbZee8KOpBCSNPRgauucjw==", "dev": true, "license": "MIT", - "peer": true, "funding": { "url": "https://github.com/sponsors/colinhacks" } diff --git a/lecture-pulse/package.json b/lecture-pulse/package.json index 94764cf..ea52941 100644 --- a/lecture-pulse/package.json +++ b/lecture-pulse/package.json @@ -12,6 +12,7 @@ "dependencies": { "@radix-ui/react-slot": "^1.2.4", "@tailwindcss/vite": "^4.1.18", + "bcryptjs": "^3.0.3", "class-variance-authority": "^0.7.1", "clsx": "^2.1.1", "framer-motion": "^12.23.26", diff --git a/lecture-pulse/src/context/AuthContext.jsx b/lecture-pulse/src/context/AuthContext.jsx index df4dae8..176523e 100644 --- a/lecture-pulse/src/context/AuthContext.jsx +++ b/lecture-pulse/src/context/AuthContext.jsx @@ -1,5 +1,6 @@ import { createContext, useContext, useState } from "react"; import { getCurrentTeacher } from "@/utils/storage"; +import bcrypt from "bcryptjs"; const AuthContext = createContext(); @@ -9,7 +10,7 @@ export function AuthProvider({ children }) { }); const [loading] = useState(false); - const login = (teacherId, password) => { + const login = async (teacherId, password) => { // Hackathon Logic: Retrieve from array of teachers or single object? // Let's assume we store a "teachers" object in LS: { [id]: { password, name } } @@ -18,7 +19,30 @@ export function AuthProvider({ children }) { const user = teachers[teacherId]; - if (user && user.password === password) { + let isValidPassword = false; + if (user) { + const isHashed = + typeof user.password === "string" && + user.password.startsWith("$2"); + if (isHashed) { + isValidPassword = await bcrypt.compare( + password, + user.password + ); + } else { + isValidPassword = user.password === password; + // Auto-migrate plaintext passwords + if (isValidPassword) { + user.password = await bcrypt.hash(password, 10); + localStorage.setItem( + "lecturePulse_teachers_db", + JSON.stringify(teachers) + ); + } + } + } + + if (user && isValidPassword) { const sessionUser = { id: teacherId, name: user.name, @@ -35,7 +59,7 @@ export function AuthProvider({ children }) { } }; - const register = (name, teacherId, password) => { + const register = async (name, teacherId, password) => { const teachersFn = localStorage.getItem("lecturePulse_teachers_db"); const teachers = teachersFn ? JSON.parse(teachersFn) : {}; @@ -43,7 +67,14 @@ export function AuthProvider({ children }) { return { success: false, message: "Teacher ID already exists." }; } - teachers[teacherId] = { name, password }; + const hashedPassword = await bcrypt.hash( + password, + 10 + ); + teachers[teacherId] = { + name, + password: hashedPassword, + }; localStorage.setItem("lecturePulse_teachers_db", JSON.stringify(teachers)); // Auto login diff --git a/lecture-pulse/src/pages/Login.jsx b/lecture-pulse/src/pages/Login.jsx index 223de50..b1892d2 100644 --- a/lecture-pulse/src/pages/Login.jsx +++ b/lecture-pulse/src/pages/Login.jsx @@ -30,12 +30,12 @@ export default function Login() { await new Promise((resolve) => setTimeout(resolve, 800)); let result; if (isLogin) { - result = login(formData.teacherId, formData.password); + result = await login(formData.teacherId, formData.password); } else { if (!formData.name) { result = { success: false, message: "Name is required." }; } else { - result = register(formData.name, formData.teacherId, formData.password); + result = await register(formData.name, formData.teacherId, formData.password); } } if (result.success) {