github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

Change Facebook oAuth param from authType to auth_type

Browse Source 
According to the Facebook docs:

https://developers.facebook.com/docs/facebook-login/manually-build-a-login-flow

The parameter should be `auth_type`, not `authType`.  A cursory look through their API history didn't show anything about it being changed, but I can confirm that using this feature does not work with `authType` in the current implementation.

Related meteor/docs#94
Related meteor/meteor#7584
Closes meteor/meteor#7078
This commit is contained in:
Jesse Rosenberger
2016-10-14 17:24:50 +03:00
committed by Ben Newman
parent 693fce59be
commit 8de559a967
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/facebook/facebook_client.js
View File

@@ -37,8 +37,8 @@ Facebook.requestCredential = function (options, credentialRequestCompleteCallbac
'&state=' + OAuth._stateParam(loginStyle, credentialToken, options && options.redirectUrl);
// Handle authentication type (e.g. for force login you need authType: "reauthenticate")
if (options && options.authType) {
loginUrl += "&authType=" + encodeURIComponent(options.authType);
if (options && options.auth_type) {
loginUrl += "&auth_type=" + encodeURIComponent(options.auth_type);
}
OAuth.launchLogin({