Improve GraphQL Data Validation and Fix HTTPX Proxy Compatibility#226
Improve GraphQL Data Validation and Fix HTTPX Proxy Compatibility#226GrekF3 wants to merge 9 commits intosubzeroid:mainfrom
Conversation
extract_direct_message already handled both new-format (xma_clip) and legacy-format (xma_media_share) Instagram clip/reel shares, but the parallel extract_reply_message only stripped the inline `clip` block. Replies that quoted reel/clip shares were getting xma_share=None on the resulting ReplyMessage even though the field existed in types.py:856. Salvaged from #226 (the only piece worth keeping — rest of the PR is either redundant with our 2.4.4 sync or regresses 0.6.6 TLS verify=True). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Thanks for the patch! Going to close this — the repo has moved a lot since November 2025 (massive sync with instagrapi 2.4.4 across 0.1.x → 0.7.x), and most of what this PR does is now either already done a different way, or actively regresses something we changed. Detailed breakdown: Already in main (different implementation)
Regressions
Obsolete
Salvaged
Considered but skipped
If you want to revisit any of the "considered" items as smaller focused PRs against current main, happy to look. Closing this one. |
Fixed incorrect behavior in HTTPX proxy handling, ensuring proper forwarding and compatibility across different proxy formats.
Improved GraphQL data guards to prevent incorrect data extraction and reduce unexpected runtime errors.
Adjusted internal validation logic to ensure safer and more predictable processing of GraphQL responses.
These changes improve reliability when working with proxies and enhance robustness of GraphQL-related operations.