docs: Added Cookie section#580
Conversation
Expanded the proxy usage documentation to include authentication, per-request proxies with custom headers, and clarified Unix socket proxy usage.
Updated the proxy usage example to reflect the new Client initialization with a list of proxies.
Fix formatting of bullet points in quickstart.md
More context added. And anchoring to API references for easy navigation between classes and models. To learn how to use it better
Updated examples for GET and POST requests, including form data and headers. Improved clarity and consistency in code snippets.
Updated response headers section to reference HeaderMap object and changed example to use .get() method.
This document explains how to handle HTTP cookies in wreq, including creating cookies, reading cookies from responses, sending cookies with requests, and managing a persistent cookie store.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughA new "Cookies" guide page was added explaining Cookie object creation/inspection, reading server-set cookies via Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/source/guide/cookie.md`:
- Around line 81-90: The example async snippet is not runnable as-is because it
omits imports and the call to run the coroutine; add top-level imports (import
asyncio and from wreq import Client) before the async def main() and invoke the
coroutine at the end with asyncio.run(main()), keeping the existing async def
main() and Client usage so the snippet becomes copy/paste executable.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 950dc15c-6513-43fe-a5c0-5120c69acb6f
📒 Files selected for processing (1)
docs/source/guide/cookie.md
Summary by CodeRabbit