From ae397a363210a19602276f21dfe7367124fd695e Mon Sep 17 00:00:00 2001
From: Jan Dvorak <storyteller@freedombase.net>
Date: Fri, 15 Mar 2019 16:08:01 +0100
Subject: [PATCH] Fix #10428 facebook permissions (#10429)

* Fix #10428 facebook permissions :passport_control:

* Add missing semicolon in tools/upgraders.js
---
 packages/facebook-oauth/facebook_server.js | 3 +--
 tools/upgraders.js                         | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/packages/facebook-oauth/facebook_server.js b/packages/facebook-oauth/facebook_server.js
index b8bfe1ddfc..db0493e980 100644
--- a/packages/facebook-oauth/facebook_server.js
+++ b/packages/facebook-oauth/facebook_server.js
@@ -5,8 +5,7 @@ Facebook.handleAuthFromAccessToken = (accessToken, expiresAt) => {
   // include basic fields from facebook
   // https://developers.facebook.com/docs/facebook-login/permissions/
   const whitelisted = ['id', 'email', 'name', 'first_name', 'last_name',
-    'middle_name', 'name_format', 'picture', 'short_name', 'age_range',
-    'birthday', 'friends', 'gender', 'hometown', 'link', 'location', 'locale'];
+    'middle_name', 'name_format', 'picture', 'short_name'];
 
   const identity = getIdentity(accessToken, whitelisted);
 
diff --git a/tools/upgraders.js b/tools/upgraders.js
index 322bdfc5dc..a581c50dfe 100644
--- a/tools/upgraders.js
+++ b/tools/upgraders.js
@@ -277,7 +277,7 @@ be removed if there is no need for the Blaze configuration interface.`,
 
       // If `<service>`, it changes to `<service>-oauth`
       if (packagesFile.getConstraint(service)) {
-        packagesFile.removePackages([service])
+        packagesFile.removePackages([service]);
         packagesFile.addPackages([`${service}-oauth`]);
       }
     });