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

Remove unecessary option fields. #42

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

Conversation

j4rs
Copy link
Contributor

@j4rs j4rs commented May 23, 2018

Last changes on LinkedIn break your code if the setup uses secure_image_url: true that anyway it is not necessary as LinkedIn always sends back the pictureUrl attribute as https. This PR intents to simplify LinkedIn oauth strategy by getting rid of that unique option field.

@j4rs j4rs changed the title Remove unecessary options fields. Remove unecessary option fields. May 23, 2018
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