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

Debug mode #170

Merged
merged 4 commits into from
May 31, 2013
Merged

Debug mode #170

merged 4 commits into from
May 31, 2013

Conversation

lepture
Copy link
Collaborator

@lepture lepture commented May 31, 2013

Add a debug mode with environment variable: DEBUG.

This is useful when doing the testing. Since I am using oauthlib's client to test the provider, everything is over http.

This patch will be helpful for testing.

https://github.com/lepture/flask-oauthlib/issues/1

@ib-lundgren
Copy link
Collaborator

Can see this being useful. Could you add that to the two other https checks in parameters.py? And maybe extract the check into its own function?

This should be added to the docs at some point too. I can do this when/if I decide to refactor part of the client API to be more consistent with the provider API.

@lepture
Copy link
Collaborator Author

lepture commented May 31, 2013

@ib-lundgren Added a is_secure_transport function in utils.

@ib-lundgren
Copy link
Collaborator

Looks good =)

ib-lundgren added a commit that referenced this pull request May 31, 2013
@ib-lundgren ib-lundgren merged commit 229b2f7 into oauthlib:master May 31, 2013
@lepture lepture deleted the debug-mode branch May 31, 2013 09:31
lepture added a commit to lepture/oauthlib that referenced this pull request May 31, 2013
ib-lundgren added a commit that referenced this pull request May 31, 2013
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

Successfully merging this pull request may close these issues.

None yet

2 participants