Skip to content
This repository has been archived by the owner on Feb 25, 2019. It is now read-only.

Fixes for invalid token and collects email, etc. #360

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixes for invalid token and collects email, etc. #360

wants to merge 1 commit into from

Conversation

cavyio
Copy link

@cavyio cavyio commented Mar 30, 2017

  • using a specific version of the API rather than the "oldest available".
  • removed the x-www form encoding thing for the token parser .. facebook doesn't use that anymore since it returns json.
  • added all of the missing files that should be there. Requires provider config for facebook to have "scope": ["email"] in your configuration.
  • set a specific API version to use, otherwise it uses the oldest, which may become deprecated in the future and newer versions may break this provider file.

@cavyio
Fixes for invalid token and collects email, etc.
563d772 
- using a specific version of the API rather than the "oldest available".
- removed the x-www form encoding thing for the token parser .. facebook doesn't use that anymore since it returns json.
- added all of the missing files that should be there. Requires provider config for facebook to have "scope": ["email"] in your configuration.
- set a specific API version to use, otherwise it uses the oldest, which may become deprecated in the future and newer versions may break this provider file.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.75% when pulling 563d772 on cavyio:patch-1 into 9ed00d5 on anvilresearch:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cavyio @coveralls