Refactor Logger: Export SENSITIVE_KEYS for Reuse - #8350
Conversation
|
This pull request addresses issue #8332 by refactoring the logger to export SENSITIVE_KEYS for reuse. This change is expected to improve code maintainability and reduce duplication. Please note that similar past decisions, such as the renaming of overlapping themes for clarity and the documentation of null handling in APIs, highlight the importance of clear naming and documentation in our codebase. Thank you for your contribution, and feel free to reach out if you have any questions! |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Exporting SENSITIVE_KEYS as a typed const array for reuse — good security-aware refactoring. CI failing — please fix.
Labels: level:beginner, quality:clean, type:refactor, type:security, mentor:Aamod007
Aamod007
left a comment
There was a problem hiding this comment.
Re-reviewing after noting the only CI failure is Vercel authorization (not code-related). All other checks (Format · Lint · Typecheck · Test, Production Build, CodeQL) pass. Approving — the Vercel deploy auth is a repo-level config issue, not a code problem.
Summary of What Has Been Done
This PR addresses issue #8332.
See the corresponding issue for full details.
Closes #8332
Note: Please assign this PR to the
tmdeveloper007account.