From 4e7b939e9320d36ccc056965454ba15f4cb8cd28 Mon Sep 17 00:00:00 2001 From: Rafi Arrafif Date: Mon, 8 Sep 2025 23:11:42 +0700 Subject: [PATCH] :lock: (security) remove disableDevtools Removed disableDevtools from the security features because it was redundant with the popup feature, which caused the authentication popup to always close. --- bun.lock | 3 --- features/auth/pages/LoginPage.tsx | 3 --- features/auth/pages/SignupPage.tsx | 4 ---- package.json | 1 - 4 files changed, 11 deletions(-) diff --git a/bun.lock b/bun.lock index cef8d5c..978edb4 100644 --- a/bun.lock +++ b/bun.lock @@ -10,7 +10,6 @@ "@tailwindcss/postcss": "^4.1.11", "commitizen": "^4.3.1", "cz-emoji": "^1.3.2-canary.2", - "disable-devtool": "^0.3.9", "framer-motion": "^12.23.3", "ky": "^1.8.2", "next": "15.3.5", @@ -1061,8 +1060,6 @@ "detect-libc": ["detect-libc@2.0.4", "", {}, "sha512-3UDv+G9CsCKO1WKMGw9fwq/SWJYbI0c5Y7LU1AXYoDdbhE2AHQ6N6Nb34sG8Fj7T5APy8qXDCKuuIHd1BR0tVA=="], - "disable-devtool": ["disable-devtool@0.3.9", "", {}, "sha512-WHCpC8f93Cn2DnTeaq57NFLcdw4921sovh1ammnp/2o8Snb704HK/Vw1/D2D1Pihc0zc8/mVKb5nchHtoadObQ=="], - "doctrine": ["doctrine@2.1.0", "", { "dependencies": { "esutils": "^2.0.2" } }, "sha512-35mSku4ZXK0vfCuHEDAwt55dg2jNajHZ1odvF+8SSr82EsZY4QmXfuWso8oEd8zRhVObSN18aM0CjSdoBX7zIw=="], "dotenv": ["dotenv@16.6.1", "", {}, "sha512-uBq4egWHTcTt33a72vpSG0z3HnPuIl6NqYcTrKEg2azoEyl2hpW0zqlxysq2pK9HlDIHyHyakeYaYnSAwd8bow=="], diff --git a/features/auth/pages/LoginPage.tsx b/features/auth/pages/LoginPage.tsx index c61dec4..ae789b6 100644 --- a/features/auth/pages/LoginPage.tsx +++ b/features/auth/pages/LoginPage.tsx @@ -25,9 +25,6 @@ const LoginPage = () => { ); useEffect(() => { - // Prevent opening devtools while in authentication page - // disableDevtool(); - /** * Check if the window has an opener (i.e., it was opened by another window) * If it does, the security checkup has passed. diff --git a/features/auth/pages/SignupPage.tsx b/features/auth/pages/SignupPage.tsx index 96b8f43..c4f4215 100644 --- a/features/auth/pages/SignupPage.tsx +++ b/features/auth/pages/SignupPage.tsx @@ -4,7 +4,6 @@ import { redirect } from "next/navigation"; import React, { useEffect, useState } from "react"; import SecurityCheckup from "@/shared/auth/ui/SecurityCheckup"; import SecurityCheckupFailed from "@/shared/auth/ui/SecurityCheckupFailed"; -import disableDevtool from "disable-devtool"; import Signup from "../ui/Signup"; const SignupPage = () => { @@ -26,9 +25,6 @@ const SignupPage = () => { ); useEffect(() => { - // Prevent opening devtools while in authentication page - disableDevtool(); - /** * Check if the window has an opener (i.e., it was opened by another window) * If it does, the security checkup has passed. diff --git a/package.json b/package.json index 875d641..d9863a7 100644 --- a/package.json +++ b/package.json @@ -19,7 +19,6 @@ "@tailwindcss/postcss": "^4.1.11", "commitizen": "^4.3.1", "cz-emoji": "^1.3.2-canary.2", - "disable-devtool": "^0.3.9", "framer-motion": "^12.23.3", "ky": "^1.8.2", "next": "15.3.5",