fix: ContentType header should be Content-Type in auth.ts #1985#1994
fix: ContentType header should be Content-Type in auth.ts #1985#1994Waqasabid99 wants to merge 1 commit intolingodotdev:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughFixed HTTP header name from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Thanks for your interest in contributing! Before we can accept this PR, you'll need to be assigned to issue #1985 first. Please request assignment on the issue, or look for other unassigned issues that you can work on. |
|
Thank you for your contribution! However, this PR references issue #1985, which is currently assigned to another contributor. To contribute, please either get assigned to an issue first or find an unassigned issue to work on. You can find unassigned issues at https://github.com/lingo-dev/lingo.dev/issues?q=is%3Aissue+is%3Aopen+no%3Aassignee |
Summary
Fix invalid ContentType header in auth.ts by replacing it with the correct "Content-Type" HTTP header.
Changes
Replaced ContentType with "Content-Type" in packages/cli/src/cli/utils/auth.ts
Ensured consistency with existing correct usage in observability.ts
Aligns with HTTP standard header naming
Testing
Business logic tests added:
Verified Headers behavior in Node — ContentType does not resolve as "Content-Type"
Confirmed "Content-Type" resolves correctly via Headers.get()
All tests pass locally
Additional verification:
Confirmed header is now correctly sent as content-type instead of contenttype
Visuals
N/A — no UI changes
Updated only Headers:

Checklist
Changeset added (not required — bug fix only, no public API changes)
Tests cover business logic (header correctness validation)
No breaking changes
Closes #1985
Summary by CodeRabbit