jaredhanson / passport-facebook

Facebook authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-facebook/?utm_source=github&utm_medium=referral&utm_campaign=passport-facebook&utm_content=about
MIT License
1.3k stars 446 forks source link

Warning message advising to let SDK handle dialog #132

Open murdockcrc opened 9 years ago

murdockcrc commented 9 years ago

Hi,

I am using the passport-facebook with default values:

When a user tries to login to my site, they get the following warning on the facebook screen (this happens only to users set as admins of the facebook app and not to the general population):

app.get('/auth/facebook', passport.authenticate('facebook', { scope: ['email', 'public_profile'] }));

You are using a display type of 'page' in a small browser window or popup. For a better user experience, show this dialog with our JavaScript SDK without specifying an explicit display type. The SDK will choose the best display type for each environment. Alternatively, use display type 'popup' if you have special requirements precluding you from using the SDK. This message is only visible to developers of your application.

I understand passport-facebook has a display parameter, but its possible combinations are not clear in the documentation, nor is it clear how to circumvent this warning (there is a link to FB's documentation, but the link does not take you anymore to content related to dialog configuration).

deveshgoyal commented 8 years ago

same here.... please give solution!

harlandjp commented 7 years ago

Anyone found a solution for this?

andrey-hohlov commented 7 years ago

https://github.com/jaredhanson/passport-facebook/issues/29 https://github.com/feathersjs/feathers-authentication-oauth2/issues/32 https://github.com/jaredhanson/passport-facebook/issues/156

noumandilshad commented 5 years ago

solution?

hamdilouati commented 3 years ago

You can force the authorization url by display popup

authorizationURL: 'https://www.facebook.com/login.php?display=popup'