Skip to content

Conversation

@matthewlmcclure
Copy link

@mitsuhiko We had a brief conversation on Twitter.

Here's a proof of concept using oauthlib via requests.

A .ftcurlish.json: https://gist.github.com/4158445.

And a demonstration using Twitter: https://gist.github.com/4158502.

This is obviously not ready to merge. Just starting a conversation. Requests has a nice high level API, but I imagine you might not like that dependency. Maybe using oauthlib directly to sign requests and then passing its output to curl would be better.

What do you think?

@matthewlmcclure
Copy link
Author

@mitsuhiko

I eliminated the earlier dependency on Requests. The pull request as of this comment depends directly on oauthlib for OAuth1 signing.

Here's an updated .ftcurlish.json showing how to configure Twitter and LinkedIn: https://gist.github.com/fe80848275e38350e60a

I tested it with both:

$ python curlish.py https://api.twitter.com/1.1/statuses/home_timeline.json
...

$ python curlish.py 'https://api.linkedin.com/v1/people/~?format=json'
...

What do you think?

@mitsuhiko
Copy link
Contributor

Is there a way to remove the other dependency as well and directly include the necessary functionality into that script? The advantage would be that it can be used without a virtualenv then which is an advantage for helper scripts.

@matthewlmcclure
Copy link
Author

I wouldn't want to reimplement oauthlib's functionality. But I could
probably make the dependency optional unless the user wants to use OAuth 1
APIs. Requests does something similar already.

On Dec 5, 2012, at 6:05 AM, Armin Ronacher notifications@github.com wrote:

Is there a way to remove the other dependency as well and directly include
the necessary functionality into that script? The advantage would be that
it can be used without a virtualenv then which is an advantage for helper
scripts.


Reply to this email directly or view it on
GitHubhttps://github.com//pull/10#issuecomment-11036442.

@matthewlmcclure
Copy link
Author

I made the oauthlib dependency optional. It's only required in case you want to use OAuth 1 authorization now.

Care to pull this into your master branch?

@dequis
Copy link

dequis commented Mar 25, 2014

Just sent a pull request to @matthewlmcclure about this pull request.

matthewlmcclure#1

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