feat(oauth): add Instagram OAuth provider#209
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
📝 WalkthroughWalkthroughAdds a new Instagram OAuth provider module exporting an ChangesInstagram OAuth Provider Addition
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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.
🧹 Nitpick comments (1)
packages/core/src/oauth/instagram.ts (1)
40-40: 🎯 Functional Correctness | 🔵 Trivial | 💤 Low valueRemove
/meendpoint doesn’t expose that field, so this query param is dead;profile.emailshouldn’t be added here either.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/core/src/oauth/instagram.ts` at line 40, Remove the unsupported email field from the Instagram /me request in the Instagram OAuth configuration, and do not map or populate profile.email from this response. Update the userInfo URL in the relevant Instagram OAuth setup so it only requests fields actually returned by the endpoint, and verify any parsing logic tied to the Instagram profile response no longer expects email.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@packages/core/src/oauth/instagram.ts`:
- Line 40: Remove the unsupported email field from the Instagram /me request in
the Instagram OAuth configuration, and do not map or populate profile.email from
this response. Update the userInfo URL in the relevant Instagram OAuth setup so
it only requests fields actually returned by the endpoint, and verify any
parsing logic tied to the Instagram profile response no longer expects email.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 1fe0b746-1c9f-47ed-9a33-5e0f69731815
📒 Files selected for processing (3)
packages/core/CHANGELOG.mdpackages/core/src/oauth/huggingface.tspackages/core/src/oauth/instagram.ts
Description
This pull request adds the
InstagramOAuth provider to the list of supported OAuth integrations in@aura-stack/auth.The provider integrates with the existing OAuth infrastructure and can be configured through the
oauthoption increateAuth().Usage
Note
This pull request is currently marked as Draft because the Instagram OAuth integration has not yet been fully validated.
During implementation, it became apparent that the Instagram authentication flow depends heavily on Meta's platform configuration and documentation, which is spread across multiple products and is not always consistent. As a result, the provider implementation cannot be fully verified until a complete application setup and authentication flow can be successfully completed.
The PR will be marked as ready for review once the provider has been validated against a working Instagram application.