-
Notifications
You must be signed in to change notification settings - Fork 176
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
FB oAuth 2.0 update breaks plugin #40
Comments
Looks like a problem with cross domain communication: Unsafe JavaScript attempt to access frame with URL https://s-static.ak.fbcdn.net/connect/xd_proxy.php?version=3#cb=f15e82491c&origin=http%3A%2F%2Flocalhost.localdomain%3A8080%... from frame with URL http://localhost.localdomain:8080/. Domains, protocols and ports must match. This will be a long night 4 me ;) when and If found a solution I'll share here |
Same here haha. |
On /views/helpers/facebook.php:416 you should see:
If you change that to this:
Then the fblogin will be able to work, although I'm afraid as to how long that will be able to work since Facebook is now Mandating that everyone use OAUTH to login and logout. |
Tnx, I needed quick fix as site was unusable atm :) |
Hopefully we get a permanent fix soon... |
I'm working on a permanent resolution. |
Updated to 3.1.1 in master -- merged php_sdk_3 after update to newest PHPSDK. Working in dev and production demo: http://facebook.webtechnick.com Please update your applications accordingly. Enjoy! CakePHP 2.0 development is still going strong, hope to have a RC soon. |
We're using this plugin on a handful of sites in Asia. We're not terribly familiar w/ the Facebook APIs/SDKs on a technical level. Will very much appreciate the permanent fix when it comes. Thanks for maintaining this code. :) |
Permanent fix is already deployed, problem was with old FB SDK. Just update your code from master branch |
With today's Facebook update all apps that are using this plugins Facebook Login is broken, including the demo on facebook.webtechnick.com.
This needs to be fixed as it cripples many sites.
The text was updated successfully, but these errors were encountered: