Skip to content
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

Flask example fails to complete connection to Github #22

Closed
mikepurvis opened this issue Aug 30, 2013 · 3 comments
Closed

Flask example fails to complete connection to Github #22

mikepurvis opened this issue Aug 30, 2013 · 3 comments

Comments

@mikepurvis
Copy link

I believe I have it set up correctly—it redirects me as expected to Github to sign in, and then github forwards my browser back to the app, where I land on a 500. There's no python error or backtrace shown either in the browser or command line—just this sequence:

10.25.0.217 - - [30/Aug/2013 15:48:46] "GET / HTTP/1.1" 200 -
10.25.0.217 - - [30/Aug/2013 15:48:52] "GET /login/github/ HTTP/1.1" 302 -
10.25.0.217 - - [30/Aug/2013 15:48:53] "GET /complete/github/?code=7900ff5b...ae1c&redirect_state=n6tqyniCa...s9tf8yXE0&state=n6tqyniCaD....tf8yXE0 HTTP/1.1" 500 -

As I've already added the app on Github at this point, it's an immediate turnaround, but I see the same error when I have to click through manually.

Note: I'm also seeing a problem where the initial pageload triggers a "'Session' object has no attribute '_model_changes'" from what looks to be some SQLAlchemy initialization stuff. Subsequent pageloads appear to work correctly, though, so I'm not certain that's a related issue.

@omab
Copy link
Owner

omab commented Aug 30, 2013

@mikepurvis, I've just tested it and worked as expected (last code revision), any chance the error happens on a custom pipeline?

@mikepurvis
Copy link
Author

Ah, I realized that not all of the dependencies matched the requirements.txt file exactly—I had some stuff that were newer versions. Once I did that, I got a proper stack trace for the error, which prompted me to run the syncdb command to populate the sqlite DB. I think I thought that would happen on its own.

Sorry for the false alarm!

@omab
Copy link
Owner

omab commented Sep 1, 2013

No problem, thanks for the update and closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants