Fix: Make Authorization Header Check Case-Insensitive in verifyReviewAdmin - #8750
Conversation
|
This pull request addresses a critical bug by making the Authorization header check case-insensitive, which is essential for proper authentication. This change has been validated in past decisions, ensuring that clients with different casing can authenticate correctly without causing runtime errors. Thank you for your contribution, and please let me know if you need any further assistance! |
📦 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.
Great attention to detail. HTTP headers are technically case-insensitive, so calling .toLowerCase()\ on both the incoming authorization header and the expected 'Bearer' string in \lib/review-admin-auth.ts\ ensures maximum compatibility with different clients and proxies. Thanks for making this robust!
|
🎉 Congratulations @tmdeveloper007! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
Made the Authorization header comparison case-insensitive in
lib/review-admin-auth.ts.Pillar
Checklist
CONTRIBUTING.mdfile.Note
Please assign this PR to the
tmdeveloper007account.Closes #8677