Skip to content

Update scopes For SSO Login#1343

Merged
atymic merged 2 commits intoSocialiteProviders:masterfrom
SanthoshSivan-Dev:patch-1
Apr 8, 2025
Merged

Update scopes For SSO Login#1343
atymic merged 2 commits intoSocialiteProviders:masterfrom
SanthoshSivan-Dev:patch-1

Conversation

@SanthoshSivan-Dev
Copy link
Copy Markdown
Contributor

@SanthoshSivan-Dev SanthoshSivan-Dev commented Mar 5, 2025

Meta changed the scope for Instagram from user_profile to 'instagram_business_basic', 'instagram_business_content_publish', 'instagram_business_manage_messages', and 'instagram_business_manage_comments'. I tested it on my local machine, and it works fine with the new scope. Can you merge my proposed changes?

meta developer doc link for your reference
Scope Update

Meta changed the scope for Instagram from `user_profile` to `'instagram_business_basic'`, `'instagram_business_content_publish'`, `'instagram_business_manage_messages'`, and `'instagram_business_manage_comments'`. I tested it on my local machine, and it works fine with the new scope. Can you merge my proposed changes?
@SanthoshSivan-Dev SanthoshSivan-Dev changed the title Update scopes Update scopes For SSO Login Mar 5, 2025
@marcelaodev
Copy link
Copy Markdown

marcelaodev commented Apr 3, 2025

I can reproduce old scope "user_profile" no longer working, so latest version of Instagram provider is altogether broken.
But I can't support this merge as well. The scopes should be only "instagram_business_basic".
The scopes set on this PR are just not needed for SSO, like "manage comments", "manage messages" and "publish content"

image

how it looks like with only "instagram_business_basic":

image

@SanthoshSivan-Dev
Copy link
Copy Markdown
Contributor Author

Hi @marcelaodev ,
Thanks for the reply. I checked using the instagram_business_basic scope, and it’s working fine now. I’ve adjusted the code—can you please check and merge this PR?

@marcelaodev
Copy link
Copy Markdown

Hi @marcelaodev , Thanks for the reply. I checked using the instagram_business_basic scope, and it’s working fine now. I’ve adjusted the code—can you please check and merge this PR?

works fine.
I'm not maintainer though 😅

@atymic atymic merged commit 089b0ae into SocialiteProviders:master Apr 8, 2025
@SanthoshSivan-Dev
Copy link
Copy Markdown
Contributor Author

@atymic , Thanks for merging the PR. But when I run composer update socialiteproviders/instagram, it doesn't update — it's still on version 5.0. Could you please check if there's a new tag or something else that needs to be create for this?

@atymic
Copy link
Copy Markdown
Member

atymic commented Apr 8, 2025

Tagged :)

@SanthoshSivan-Dev
Copy link
Copy Markdown
Contributor Author

@atymic Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants