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

Back-pagination spinner gets stuck after using Purge History API in synapse #2566

Closed
4nd3r opened this issue Nov 10, 2016 · 11 comments
Closed

Comments

@4nd3r
Copy link
Contributor

4nd3r commented Nov 10, 2016

not sure if synapse or vector-web bug.

scrot-1478804473

@richvdh
Copy link
Member

richvdh commented Nov 10, 2016

It's trying to back-paginate. Is this in a federated room, or one that only exists on your server?

@4nd3r
Copy link
Contributor Author

4nd3r commented Nov 10, 2016

only in my server

@richvdh
Copy link
Member

richvdh commented Nov 10, 2016

sounds like a synapse bug then. Do you get the same behaviour after reloading riot?

@4nd3r
Copy link
Contributor Author

4nd3r commented Nov 10, 2016

yes

@richvdh richvdh changed the title Wheel stays spinning after using Purge History API in synapse Back-pagination spinner gets stuck after using Purge History API in synapse Nov 10, 2016
@richvdh
Copy link
Member

richvdh commented Nov 10, 2016

Yup, repeated here; synapse is returning 500 errors. Raised matrix-org/synapse#1623.

@4nd3r
Copy link
Contributor Author

4nd3r commented Nov 10, 2016

thanks

@4nd3r 4nd3r closed this as completed Nov 10, 2016
@richvdh
Copy link
Member

richvdh commented Nov 11, 2016

Let's keep this open to track it from the application side.

@babolivier
Copy link
Contributor

We realised today that matrix-org/synapse#1623 had been fixed (just hadn't been closed at the time), should we also close this issue?

@turt2live
Copy link
Member

I think this is now waiting on matrix-org/matrix-spec-proposals#2251 to be fixed so we can fix the infinite spinner problem.

@babolivier
Copy link
Contributor

@turt2live Right, but is that infinite spinner problem really still a thing though? I've just tried to reproduce it by creating a room on my local HS (unfederated), sending multiple messages in it, purging the history from one of these messages, and hitting "Clear cache and reload" in Riot. Once the room was loaded by Riot, it apparently tried to back-paginate events before the oldest event not purged, and when /messages returned with an empty chunk ([]) it stopped hitting that endpoint and showing a spinner.

@turt2live
Copy link
Member

Hmm, indeed. Closing this as fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants