search

coto / gae-boilerplate

Google App Engine Boilerplate
https://dev-dot-sandengine.appspot.com/
Other
685 stars 189 forks source link

Change Facebook Permissions Defaults #283

Open davedargo opened 10 years ago

davedargo commented 10 years ago

Is publish_stream necessary for a facebook authentication? Can we allow a different set of facebook permissions when using facebook for authentication?

Line 260 in handlers.py, SocialLoginHandler could be changed to:

perms = self.app.config.get('fb_perms', ['email', 'publish_stream'])

without affecting current implementations.

coto commented 10 years ago

No, it is not necessary, if you want, you can add a change to define that in config.