Skip to content
This repository has been archived by the owner on May 27, 2019. It is now read-only.

Add oauth support #2

Open
lucaspiller opened this issue May 5, 2013 · 7 comments · May be fixed by #7
Open

Add oauth support #2

lucaspiller opened this issue May 5, 2013 · 7 comments · May be fixed by #7

Comments

@lucaspiller
Copy link
Owner

Twitter will be removing HTTP basic authentication from the streaming API at the beginning of June.

WIP in the oauth branch.

@pdincau
Copy link

pdincau commented Oct 8, 2013

Any news about this feature?

Thanks.

@arjan
Copy link
Contributor

arjan commented Dec 10, 2013

Ping, any update on this?

@lucaspiller
Copy link
Owner Author

No it isn't complete yet, you can see WIP so far which is basically just refactoring the existing code to allow different authentication schemes. If you already have the oAuth token, all that is required to use it is extend the method that generates the request headers:

% TODO extend for oauth
-spec headers_for_auth(term(), term(), list()) -> list().
headers_for_auth({basic, [User, Pass]}, _Endpoint, _Params) ->
generate_auth_headers(User, Pass).

I haven't had time to work on this recently, however PRs are appreciated :)

@arjan
Copy link
Contributor

arjan commented Dec 10, 2013

Yup, working on a PR as we speak :) In a few days I'll have something.

@arjan
Copy link
Contributor

arjan commented Dec 10, 2013

@lucaspiller what kind of local server are you testing again for espec?

@lucaspiller
Copy link
Owner Author

Good question. I'll have a look and see if I can figure it out...

@arjan
Copy link
Contributor

arjan commented Dec 10, 2013

FYI, https://github.com/arjan/twerl/commits/refactoring
still working on it. I need reconnect behaviour.. :)

@lucaspiller lucaspiller linked a pull request Jun 4, 2014 that will close this issue
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants