Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

linked in provider options and profile update #8345

Merged
merged 12 commits into from
Aug 29, 2023

Conversation

HugeLetters
Copy link
Contributor

☕️ Reasoning

It seems LinkedIn has updated its authentication procedure - this PR updates the provider to reflect those changes.
LinkedIn renamed required scopes(permissions) and also changed the shape of profile data.

🧢 Checklist

  • Documentation - not needed
  • Tests
  • Ready to be merged

🎫 Affected issues

Please scout and link issues that might be solved by this PR.

Fixes: #8316

📌 Resources

@vercel
Copy link

vercel bot commented Aug 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
auth-docs ❌ Failed (Inspect) Aug 29, 2023 4:17am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
next-auth-docs ⬜️ Ignored (Inspect) Visit Preview Aug 29, 2023 4:17am

@vercel
Copy link

vercel bot commented Aug 17, 2023

@HugeLetters is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added providers core Refers to `@auth/core` legacy Refers to `next-auth` v4. Minimal maintenance. labels Aug 17, 2023
@joemulberry
Copy link

would love this to be solved

@ss-zheng
Copy link

OMG I was stucked for a night in order to enable linkedin auth, and found this PR this morning. Please merge this appreciated it!

Copy link
Member

@ThangHuuVu ThangHuuVu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution, I left some comments. 🙏

packages/core/src/providers/linkedin.ts Outdated Show resolved Hide resolved
packages/next-auth/src/providers/linkedin.ts Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the legacy Refers to `next-auth` v4. Minimal maintenance. label Aug 28, 2023
@ThangHuuVu ThangHuuVu merged commit 35977bf into nextauthjs:main Aug 29, 2023
8 of 11 checks passed
@AnatoleT
Copy link

Would love to see this solved too :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Refers to `@auth/core` providers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sign In with LinkedIn is deprecated, use Sign In with LinkedIn using OpenID Connect instead
5 participants