-
Notifications
You must be signed in to change notification settings - Fork 169
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Undefined index: expires @ line 77 #7
Comments
I cant reproduce this locally, $results include the 'expires' data for me. 2013/6/20 Julien Huriez [email protected]
|
Yes, the result :
|
I can't reproduce this either. You might want to check that you're using the latest versions of Opauth and Facebook strategy. Also, give it a try at https://opauth.org/ |
Yes i have the latest versions of Opauth and Facebook Strategy. It's strange because it's work with Twitter, Google, Github and Instagram PS: I use it with FuelPHP framework (1.6) |
Looks like on some occasions this happens to other people as well: https://developers.facebook.com/bugs/339375252810525/ Even though they have closed the bug, this is a bug on facebook side, which 2013/6/20 Julien Huriez [email protected]
|
Same bug here, When the app is allowed by my facebook account, the access_token is retrieved correctly, then after a few tries the facebook response doesn't contain the "expires" field anymore. It's definitely a Facebook issue, but the library should implement a catch or something. If anyone have informations to avoid that... :) |
@tompiard I'm not sure if this is relevant: http://stackoverflow.com/questions/15923326/facebook-access-token-missing-expiry-property-after-offline-access-deprecations And also https://developers.facebook.com/roadmap/offline-access-removal/ One way to get over this as a consumer is not to rely on |
Could be related to #16 |
After login i have this error :
It's at line 77.
For info, Twitter and Google opauth's work fine
The text was updated successfully, but these errors were encountered: