fix: prevent double https:// in R2 public URLs#17
fix: prevent double https:// in R2 public URLs#17jezweb wants to merge 1 commit intoifindev:mainfrom
Conversation
The CLOUDFLARE_R2_URL environment variable already includes the protocol prefix (https://), so prepending another 'https://' results in malformed URLs like 'https://https://...'. This fix removes the hardcoded protocol prefix and uses the env var value directly. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Warning Rate limit exceeded@jezweb has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 12 minutes and 14 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
✨ Finishing touches
🧪 Generate unit tests (beta)
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 |
Problem
The
CLOUDFLARE_R2_URLenvironment variable already includes the protocol prefix (https://), as shown in the README examples:However, the code in
src/lib/r2.ts:49was prepending anotherhttps://, resulting in malformed URLs like:Solution
Remove the hardcoded
https://prefix and use the environment variable value directly.Changes
src/lib/r2.ts: Updated public URL construction to not prepend protocolCLOUDFLARE_R2_URLalready includes protocolImpact
This fixes image uploads that would have failed due to invalid URLs being stored in the database.
🤖 Generated with Claude Code