Skip to content

remove debug logs#105

Merged
akshay0611 merged 1 commit intoNsanjayboruds:mainfrom
madhav2348:remove-logs
Feb 25, 2026
Merged

remove debug logs#105
akshay0611 merged 1 commit intoNsanjayboruds:mainfrom
madhav2348:remove-logs

Conversation

@madhav2348
Copy link
Copy Markdown
Contributor

@madhav2348 madhav2348 commented Feb 24, 2026

Description

resolve #87
backend middleware contains debug logs that exploit authentication
backend/middleware/isAuth.js

Solution

remove those debug logs without changing any authentication logic

@madhav2348
Copy link
Copy Markdown
Contributor Author

@akshay0611 please review

@akshay0611 akshay0611 self-requested a review February 25, 2026 05:27
Copy link
Copy Markdown
Collaborator

@akshay0611 akshay0611 left a comment

Choose a reason for hiding this comment

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

Re-verified PR #105 against issue #87.

-backend/middleware/isAuth.js no longer logs sensitive auth data (token/user id).

  • Middleware behavior remains unchanged (same success path and same 401 responses).

This looks good to merge. Nice security cleanup.

@akshay0611 akshay0611 merged commit 4e3092b into Nsanjayboruds:main Feb 25, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SECURITY]: Remove sensitive auth debug logs from middleware

2 participants