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

update linkedin provider name to the correct one #145

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FadyBengoufa
Copy link

@FadyBengoufa FadyBengoufa commented Dec 24, 2024

Description of the issue

Following the documentation of devdojo auth package, I wanted to use linkedin. But it was not working, although google worked, so my setup was correct.

How to reproduce the error

  • Follow exactly the docs and try to create a developer app in linkedin (you need a linkedin page for that, create one)
  • Activate Share on LinkedIn and Sign In with LinkedIn using OpenID Connect products in the Products section on linkedin developer portal.
  • Add auth/linkedin/callback in the Authorized redirect URLs for your app, but it will not work because the linkedin API has changed.

Reason from what I understood: Microsoft has deprecated the old API as of August 1, 2023!

Here some of the references:

How I fixed it

I read through the linkedin api, and indeed the the API was changed instead of using "linkedin" it's now "linkedin-openid".
You need to change the callback redirect url in linkedin developers portal to auth/linkedin-openid/callback.

Also slack is now "slack-openid" although I haven't tested it, I can try and test it than update it as well.

I'm not sure about something in regards to this issue to be honest, it seems that when I activated the openid in linkedin developers (I can't disable it after), at that point it won't allow me to pass through the linkedin API with the "linkedin" provider but I need to use "linkedin-openid", but maybe If I haven't activated it, it could have worked. This issue needs to be handled so that both would work if that's the case, let me know what do you think.

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.

1 participant