search

facebookarchive / fbconsole

A micro api client for writing scripts against the Facebook Graph API.
Other
325 stars 74 forks source link

Improved configuration flexbility #22

Closed dogukantufekci closed 11 years ago

dogukantufekci commented 11 years ago

Replaced 127.0.0.1 with DOMAIN variable, and added REDIRECT_URI for flexible configuration.

pcardune commented 11 years ago

Can you update the README file too?

dogukantufekci commented 11 years ago

I was trying to make it work on a Flask Heroku app. However I ran into a brick wall with the hardcoded configuration. Then I finally gave up. So I guess my pull request is not longer valid.

I will turn back to it later to solve this issue; then sure I will update the README.

Thanks for sharing your code!

pcardune commented 11 years ago

What brick wall did you run into?

dogukantufekci commented 11 years ago

When I changed the SERVER_PORT it would not fbconsole.authenticate() properly. Redirect URL would still be 8080, thus cause problems with Facebook App url verification.

One other reason I gave up using this library was that fbconsole.authenticate() had to write a document to store ACCESS_TOKEN. I assume this would cause problems if I were to have multiple users authenticate at the same time? I am not sure if I properly understand the architecture.

pcardune commented 11 years ago

Ah. Yeah, this library is really just for small scripts you might run on the command line. It's not really set up for multiuser web applications, though it could be modified to make it work. I haven't gotten around to making the necessary changes yet.

On Apr 15, 2013, at 10:10 PM, Dogukan Tufekci notifications@github.com wrote:

When I changed the SERVER_PORT it would not fbconsole.authenticate() properly. Redirect URL would still be 8080, thus cause problems with Facebook App url verification.

One other reason I gave up using this library was that fbconsole.authenticate() had to write a document to store ACCESS_TOKEN. I assume this would cause problems if I were to have multiple users authenticate at the same time? I am not sure if I properly understand the architecture.

— Reply to this email directly or view it on GitHub.