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

Add sign out to Account #30

Open
CodedOre opened this issue Mar 9, 2022 · 2 comments
Open

Add sign out to Account #30

CodedOre opened this issue Mar 9, 2022 · 2 comments
Labels
Bug Platform: Twitter Issue specific to the Twitter platform
Milestone

Comments

@CodedOre
Copy link
Owner

CodedOre commented Mar 9, 2022

At some point we need to add a method to sign an Account out and revoke the access token connected.

Since Twitter's OAuth endpoints aren't clearly documented and it's not the highest priority now, I noting it as a to-do for later.

@CodedOre CodedOre added ToDo Good to Start Good Issue to start with if you want to contribute labels Mar 9, 2022
@CodedOre
Copy link
Owner Author

So, technically this is now implemented with Account.revoke_access.

However, the current implementation does not work with Twitter. The revoke method 1 requires the parameters such as client_id and the token to be passed on as x-www-form-urlencoded. And it seems for me like librest is encoding it as such 2 (or I missed something). Still, the servers return only a Forbidden, without any note of what I'm doing wrong.

So, since it's technically implemented by partially broken, I keep this open for now.

Footnotes

  1. https://developer.twitter.com/en/docs/authentication/oauth-2-0/user-access-token

  2. https://gitlab.gnome.org/GNOME/librest/-/blob/master/rest/rest-proxy-call.c#L898-900

@CodedOre CodedOre added Bug Platform: Twitter Issue specific to the Twitter platform and removed ToDo Good to Start Good Issue to start with if you want to contribute labels Sep 13, 2022
@IBBoard
Copy link
Contributor

IBBoard commented Sep 14, 2022

I guess you need to intercept the traffic to see what's actually being transmitted. It's possible to get Wireshark to decrypt HTTPS (I've done it with Firefox when checking some website stuff) but it looks like it's much harder to get the initial encryption key with OpenSSL.

@CodedOre CodedOre added this to the 2.0 milestone Oct 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Platform: Twitter Issue specific to the Twitter platform
Projects
None yet
Development

No branches or pull requests

2 participants