Allow setting the Referer header - #16
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Included review availability: 4 reviews are currently available. Based on recent review activity, included reviews refill at 5 per hour. 📝 WalkthroughWalkthroughThe connection version is updated to 3.1.0. Connections can store an optional Referer URL. Generated requests include the Referer header only when configured. Tests cover the default and configured behaviors. ChangesReferer Header Support
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR adds configurable Referer headers for requests and includes corresponding tests; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
Comment |
Add a
Connection::setReferer()method that sends the configured URL as the Referer header on every request, so API consumers can identify themselves for diagnostics.Summary by CodeRabbit
New Features
Chores