Skip to content

Fix bugs across ExamTime#306

Closed
Xenon010101 wants to merge 1 commit into
hereisSwapnil:mainfrom
Xenon010101:fix/various-bugs
Closed

Fix bugs across ExamTime#306
Xenon010101 wants to merge 1 commit into
hereisSwapnil:mainfrom
Xenon010101:fix/various-bugs

Conversation

@Xenon010101

Copy link
Copy Markdown

��## Summary
Fixed 7+ bugs in the ExamTime codebase:

Bug Fixes

  1. Fixed wrong model reference in deleteQuestion � used Request.findByIdAndDelete instead of Question.findByIdAndDelete; would throw ReferenceError at runtime
  2. Fixed bare catch block missing error parameter � catch block lacked parameter, causing ReferenceError
  3. Removed JWT token console leak � console.log(token) leaked auth tokens to server logs
  4. Removed user object console leak � console.log(user) leaked sensitive user data
  5. Removed debug console.log for OTP verification � leaked verification state
  6. Removed JWT decoded data console leak in verifyToken middleware
  7. Removed debug console.log in DocGrid component � leaked user object to browser console
  8. Replaced console.log with console.error in catch blocks across Navbar, UploadPage, and user.controller

@vercel

vercel Bot commented May 28, 2026

Copy link
Copy Markdown

@Xenon010101 is attempting to deploy a commit to the swapnilsingh99's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, @Xenon010101! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant