{"action":"created","issue":{"url":"https://api.github.com/repos/meteor/meteor/issues/6975","repository_url":"https://api.github.com/repos/meteor/meteor","labels_url":"https://api.github.com/repos/meteor/meteor/issues/6975/labels{/name}","comments_url":"https://api.github.com/repos/meteor/meteor/issues/6975/comments","events_url":"https://api.github.com/repos/meteor/meteor/issues/6975/events","html_url":"https://github.com/meteor/meteor/pull/6975","id":153198919,"number":6975,"title":"Remove the google `profile` scope.","user":{"login":"MasterAM","id":226666,"avatar_url":"https://avatars.githubusercontent.com/u/226666?v=3","gravatar_id":"","url":"https://api.github.com/users/MasterAM","html_url":"https://github.com/MasterAM","followers_url":"https://api.github.com/users/MasterAM/followers","following_url":"https://api.github.com/users/MasterAM/following{/other_user}","gists_url":"https://api.github.com/users/MasterAM/gists{/gist_id}","starred_url":"https://api.github.com/users/MasterAM/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MasterAM/subscriptions","organizations_url":"https://api.github.com/users/MasterAM/orgs","repos_url":"https://api.github.com/users/MasterAM/repos","events_url":"https://api.github.com/users/MasterAM/events{/privacy}","received_events_url":"https://api.github.com/users/MasterAM/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-05-05T09:36:12Z","updated_at":"2016-05-24T23:13:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/meteor/meteor/pulls/6975","html_url":"https://github.com/meteor/meteor/pull/6975","diff_url":"https://github.com/meteor/meteor/pull/6975.diff","patch_url":"https://github.com/meteor/meteor/pull/6975.patch"},"body":"This is a re-submission of https://github.com/meteor/meteor/pull/5699.\r\n\r\nThe `email` scope is enough for getting the required data, including the user ID, so no need for the `profile` scope, which may deter users.\r\n\r\nThe discussion started at https://github.com/meteor/meteor/issues/5650\r\n\r\nOur team anticipates Google to be a major choice for users during signup, so we would like to alleviate some of the concerns that requesting the `profile` scope may bring.\r\n\r\nWe would also like developers to have more control over more aspects, such as the way the popup window behaves (e.g, forcing it upon login for situations where there is more than one logged-in Google account).\r\n\r\nI would really like to hear your opinions in the discussion thread."},"comment":{"url":"https://api.github.com/repos/meteor/meteor/issues/comments/221429734","html_url":"https://github.com/meteor/meteor/pull/6975#issuecomment-221429734","issue_url":"https://api.github.com/repos/meteor/meteor/issues/6975","id":221429734,"user":{"login":"MasterAM","id":226666,"avatar_url":"https://avatars.githubusercontent.com/u/226666?v=3","gravatar_id":"","url":"https://api.github.com/users/MasterAM","html_url":"https://github.com/MasterAM","followers_url":"https://api.github.com/users/MasterAM/followers","following_url":"https://api.github.com/users/MasterAM/following{/other_user}","gists_url":"https://api.github.com/users/MasterAM/gists{/gist_id}","starred_url":"https://api.github.com/users/MasterAM/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MasterAM/subscriptions","organizations_url":"https://api.github.com/users/MasterAM/orgs","repos_url":"https://api.github.com/users/MasterAM/repos","events_url":"https://api.github.com/users/MasterAM/events{/privacy}","received_events_url":"https://api.github.com/users/MasterAM/received_events","type":"User","site_admin":false},"created_at":"2016-05-24T23:13:37Z","updated_at":"2016-05-24T23:13:37Z","body":"We could make it configurable via Meteor.settings, and have it add the `profile` scope by default.\r\nHaving an explicit setting could override it.\r\n\r\nHaving said that, there are a couple more things I would like to make configurable about the Google auth process. Would you advise to add them to this PR or create a separate one once this one is agreed upon and merged?"}}
{"actor":{"display_login":"MasterAM"}}