jaredhanson / passport-facebook

Facebook authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-facebook/?utm_source=github&utm_medium=referral&utm_campaign=passport-facebook&utm_content=about
MIT License
1.29k stars 446 forks source link

Change version facebook api #256

Closed robertgon97 closed 5 years ago

robertgon97 commented 5 years ago

Adding the latest version of facebook for oAuth requests

READ THIS FIRST!

Are you implementing a new feature? NO

Requests for new features should first be discussed on the developer forum. This allows the community to gather feedback and assess whether or not there is an existing way to achieve the desired functionality.

If it is determined that a new feature needs to be implemented, include a link to the relevant discussion along with the pull request.

Is this a security patch? NO

Do not open pull requests that might have security implications. Potential security vulnerabilities should be reported privately to jaredhanson@gmail.com. Once any vulerabilities have been repaired, the details will be disclosed publicly in a responsible manner. This also allows time for coordinating with affected parties in order to mitigate negative consequences.

If neither of the above two scenarios apply to your situation, you should open a pull request. Delete this paragraph and the text above, and fill in the information requested below.

Checklist

tiblu commented 5 years ago

@robertgon97 I think you can close this PR as passport-facebook has the change in 3.0.0 (https://github.com/jaredhanson/passport-facebook/blob/308c47eb3b0d535b0210e20a90b974337dc33232/lib/strategy.js#L50)