search

opauth / linkedin

LinkedIn strategy for Opauth
16 stars 18 forks source link

scope #3

Closed marcelosiqueira closed 7 years ago

marcelosiqueira commented 11 years ago

variable 'scope' not define in $defaults

uzyn commented 11 years ago

Scope is already supported as an optional param with no predefined default values.

I'm still unsure if we should make it default as that may seem rather intrusive as a default. Perhaps we can simply add a note at README on how to specify additional scope to obtain email.

Anyway, I gave your PR a try but I can't seem to be able to obtain my email address from it.

marcelosiqueira commented 11 years ago

Not using scope will return an error, it took myself enough to find that it was for that reason.

So I had to create this option in the array defaults.

ceeram commented 11 years ago

For me linkedin works fine without scope key, if you want it to use scope, you can define it in your strategy config