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

Remove streaming #1266

Closed
Xefir opened this issue Apr 22, 2020 · 10 comments
Closed

Remove streaming #1266

Xefir opened this issue Apr 22, 2020 · 10 comments

Comments

@Xefir
Copy link
Contributor

Xefir commented Apr 22, 2020

Twitter forbid the use of streaming, Mastodon Streaming API isn't implemented and isn't really reliable and I don't think other platform has this option.

So I think, why not remove all the code around streaming capabilities witch seems useless.
I just thinking on it, I just want your opinion on this.
Thank you :)

@Tlaster
Copy link
Contributor

Tlaster commented Apr 23, 2020

I can confirm that streaming is not working anymore with twitter. I think it's time to remove streaming.

@Tedko
Copy link
Contributor

Tedko commented Apr 23, 2020

if Twitter streaming is not usable anymore then lets rm that

@Tlaster
Copy link
Contributor

Tlaster commented Apr 23, 2020

I just notice that twitter replace streaming API with Account Activity API (migration guide), which is premium/enterprise API, should we migrate to Account Activity API?

@Xefir
Copy link
Contributor Author

Xefir commented Apr 23, 2020

Do we have access to this premium/enterprise API ? If yes, why not. If no, will you be able to afford the price of this API ?

@Tlaster
Copy link
Contributor

Tlaster commented Apr 23, 2020

No, we can remove streaming until someone willing to provide premium/enterprise key

@Tedko
Copy link
Contributor

Tedko commented Apr 23, 2020

Enterprise API so far only allow up to 250 user etc. not really that useful ;(

@Xefir
Copy link
Contributor Author

Xefir commented Apr 23, 2020

I think, if we implement the Account Activity API, maybe with the hack with the official keys, this will maybe works because normally, official keys isn't limited don't they ?

BTW, I don't understand their concept of webhooks at all T_T

@Tlaster
Copy link
Contributor

Tlaster commented Apr 24, 2020

The official key support is not that stable, 5052d1f might happen again

@Tlaster
Copy link
Contributor

Tlaster commented May 4, 2020

I think we can disable it for serval versions and tell the user streaming is not working anymore since twitter removes the streaming API, then remove it completely.

Tlaster added a commit that referenced this issue May 12, 2020
@Tlaster
Copy link
Contributor

Tlaster commented May 12, 2020

Remove Streaming API since there is no replacement API which is useable for this.
Will put into the changelog

@Tlaster Tlaster closed this as completed May 12, 2020
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

No branches or pull requests

3 participants