search

SoapSeller / omniauth-facebook-access-token

51 stars 68 forks source link

Support for expires_at, expires_in params #3

Closed gmcnaughton closed 11 years ago

gmcnaughton commented 11 years ago

I would like to be able to provide expires_at or expires_in along with access_token and have it get passed through to the OAuth2::AccessToken. Otherwise after signup or login, request.env["omniauth.auth"].credentials.expires_at is always nil.

I'm using the Facebook mobile SDK and the Cordova Facebook Plugin. When my app gets called back by the Facebook mobile SDK, both accessToken, expiresAt, and expirationTime are available. I can pass access_token along, but expiresAt/expirationTime currently get lost.

SoapSeller commented 11 years ago

Your fix looks good (:

gmcnaughton commented 11 years ago

@SoapSeller thanks for the quick release! :)