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

Tb/python lib rev #1

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Tb/python lib rev #1

wants to merge 12 commits into from

Conversation

tonyblank
Copy link

Hey everyone!

I've done a bit of polishing on the python lib. Any comments would be great! I'll be releasing it next week.

Thanks,
Tony

kkinder and others added 12 commits October 9, 2011 18:02
…r example, when it doesn't like your oath credentials, it returns the following: {u'code': 105, u'type': u'error', u'value': u'invalid consumer (REDACTED)'}


This change will show this error in the traceback instead of a more cryptic KeyError.
context.io appears to return certain errors without a message key. For ex
@dbgrandi
Copy link

Nice, this fixes an issue with calling File.get_content() that I just ran into.

The old code threw an error if it could not decode a response as json, which fails when calling get_content() except in the rare cases where people have a json file attached to a message. :)

@tonyblank
Copy link
Author

Hey dbgrandi! Happy to hear this helps you out! Let me know if you have any other comments or questions.

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.

3 participants