-
Notifications
You must be signed in to change notification settings - Fork 21
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
500 error logging in #7
Comments
Here is the error message coming back from the server
|
It seems that your docker machine can't access the outer world (facebook.com more specifically). I'm not a specialist in docker. @andreimarinescu can you have a look ? :) |
I resolved this issue by switching my VM from NAT network to Bridged Network. I'm not 100% sure, but it seems to me that NAT would be the correct setup for a cloud deployment like AWS. However, I'm not sure it works for OSX local deployment. |
I'm not sure that is the correct solution since I'm not getting the 500 error again. |
I have to restart both the backend and worker/api using docker-compose to stop the 500 error. Now I get this error:
Which seems like an error with my Facebook app setup, however my app is set up correctly and my email is publicly available, so not sure why that error is occuring. |
perhaps the issue is that the API doesn't explicitly request the "email" field. I noticed on line 85 of the users.js API the code reads:
Perhaps this should read the following.
|
That just tells facebook you want those respective fields, if they are missing (in your case, email address) you're not getting it anyway. The access_token you give to the API dictates the permissions. You should look how the access_token is requested. |
Actually, in v2.5 of the Facebook API, you need to explicitly request the email field. You can test this in the Facebook Graph explorer. |
Well, if it works you can commit the changes and make a pull request 👍 |
will do 👍 👍 |
Thanks, @aaronfranco. This change actually sounds reasonable from my point of view, let us know if we can further help in testing this out. |
Cool, thanks! I think the issue here is that new users can only deploy Facebook v2.5 apps. Even if v2.3 is requested, Facebook automatically upgrades the request to v2.5. This makes it impossible for any new users to properly test the demoApp successfully because the demo and API are built with v2.3 which is now depreciated for all new Facebook apps. |
@aaronfranco Yes, you can use the shared docker compose file along with a cloned copy of telepat-api. You can also edit the API out of the telepat docker compose file, and just run the workers via docker. Hope this helps! |
Thanks! Sorry about that. I realized that solution after I posted the comment, then I deleted the comment. |
Should be fixed now, the cause was human error 😝 . I was referring to the wrong variable when looking for the email field. I included those fields like you suggested. Thanks a lot ! 😀 |
I have the demoApp running on docker. When I log in with Facebook, the Telepat-api server log shows the following error.
Then, on the front end, it results in this console error output.
When I list my apps, here are the credentials that I see and use in my demoApp.
Please let me know if I'm missing anything.
The text was updated successfully, but these errors were encountered: