-
Notifications
You must be signed in to change notification settings - Fork 9
Feat auth frontend #115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat auth frontend #115
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Sweep: PR ReviewAuthors of pull request: @SteveStef, @Laphatize .env.developmentAdded a new environment variable src/components/AuthPopup.jsxRemoved Firebase authentication logic, causing the component to always display the "User logged out" state. Sweep Found These Issues
CTFGuideNext/src/components/AuthPopup.jsx Lines 3 to 24 in 588a58f
View Diff src/components/StandardNav.jsxRemoved Firebase authentication and replaced the logout mechanism with cookie manipulation and router redirection. Sweep Found These Issues
CTFGuideNext/src/components/StandardNav.jsx Lines 51 to 53 in 588a58f
View Diff src/components/groups/assignments/create-challenge.jsxReplaced Firebase authentication with JWT token authentication using cookies. Sweep Found These Issues
CTFGuideNext/src/components/groups/assignments/create-challenge.jsx Lines 19 to 24 in 588a58f
View Diff CTFGuideNext/src/components/groups/assignments/create-challenge.jsx Lines 89 to 91 in 588a58f
View Diff src/components/groups/assignments/updateChallengeInfo.jsxReplaced Firebase authentication with JWT-based authentication by modifying import statements and token retrieval logic. Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.
View Diff
View Diff src/components/onboarding/DataAsk.jsxSimplified the logout function, modernized the HTTP request to use Fetch API, and improved code readability and maintainability through various formatting changes. src/components/onboarding/OnboardingFlow.jsxModified Sweep Found These Issues
CTFGuideNext/src/components/onboarding/OnboardingFlow.jsx Lines 7 to 30 in 588a58f
View Diff src/components/studio/forking/editor.jsxSwitched authentication from Firebase to JWT tokens stored in cookies. Sweep Found These Issues
CTFGuideNext/src/components/studio/forking/editor.jsx Lines 49 to 55 in 588a58f
View Diff src/config/firebaseConfig.jsRemoved the Firebase configuration and initialization code by deleting Sweep Found These Issues
CTFGuideNext/src/config/firebaseConfig.js Lines 1 to 60 in 2fcae20
View Diff src/config/jwt.jsRemoved the entire JWT configuration and token management functionality from Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.
CTFGuideNext/src/config/jwt.js Lines 1 to 34 in 2fcae20
View Diff src/config/lessonConfigs.jsAdded a newline character at the end of the file for proper formatting. src/middleware.jsRemoved two unnecessary blank lines in src/pages/_app.jsxIntegrated Google OAuth by wrapping the main component with Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.
CTFGuideNext/src/pages/_app.jsx Line 7 in 588a58f
View Diff src/pages/create/new.jsxReplaced Firebase authentication with cookie-based JWT decoding for token retrieval in file uploads. Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.
CTFGuideNext/src/pages/create/new.jsx Lines 69 to 72 in 588a58f
View Diff CTFGuideNext/src/pages/create/new.jsx Lines 69 to 72 in 588a58f
View Diff CTFGuideNext/src/pages/create/new.jsx Lines 70 to 72 in 588a58f
View Diff CTFGuideNext/src/pages/create/new.jsx Lines 70 to 72 in 588a58f
View Diff src/pages/login.jsxReplaced Firebase authentication with custom API calls and integrated Google login using Sweep Found These Issues
CTFGuideNext/src/pages/login.jsx Lines 41 to 49 in 588a58f
View Diff CTFGuideNext/src/pages/login.jsx Lines 52 to 63 in 588a58f
View Diff CTFGuideNext/src/pages/login.jsx Lines 65 to 68 in 588a58f
View Diff Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.
CTFGuideNext/src/pages/login.jsx Lines 17 to 38 in 588a58f
View Diff src/pages/register.jsxReplaced Firebase authentication with custom email existence check and Google OAuth login, updated registration form validation, and added conditional rendering for onboarding flow. Sweep Found These Issues
CTFGuideNext/src/pages/register.jsx Lines 79 to 101 in 588a58f
View Diff CTFGuideNext/src/pages/register.jsx Lines 79 to 101 in 588a58f
View Diff CTFGuideNext/src/pages/register.jsx Lines 103 to 116 in 588a58f
View Diff CTFGuideNext/src/pages/register.jsx Lines 118 to 121 in 588a58f
View Diff Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.
CTFGuideNext/src/pages/register.jsx Lines 79 to 101 in 588a58f
View Diff CTFGuideNext/src/pages/register.jsx Lines 103 to 116 in 588a58f
View Diff src/pages/settings.jsxCommented out Firebase storage and authentication functionalities, and replaced Firebase password update logic with an API request in the Sweep Found These Issues Potential IssuesSweep isn't 100% sure if the following are issues or not but they may be worth taking a look at. src/pages/users/[user].jsxRemoved Firebase storage functionalities and the user login status check, and cleaned up unused imports and blank lines. Sweep Found These Issues
CTFGuideNext/src/pages/users/[user].jsx Lines 283 to 306 in 588a58f
View Diff The following files were not reviewed because our filter identified them as typically non-human-readable or less important files (e.g., dist files, package.json, images). If this is an error, please let us know.
|
No description provided.