matrix-static icon indicating copy to clipboard operation
matrix-static copied to clipboard

Older direct links stop working

Open xaur opened this issue 7 years ago • 7 comments

I opened this message direct link from Jul 5

https://view.matrix.org/room/!wSdymYrEpBhsWlDJuk:decred.org/?anchor=$153090455433548QNlFj:decred.org

and it showed "Some error has occurred". Then I removed the anchor part from the URL to open latest messages in the room and pressed "Load older messages" until I reached Jul 5 and it showed the message I was after. Then I visited initial direct link again and it opened.

Example of direct link that doesn't work (until someone visits it via "Load older messages"):

https://view.matrix.org/room/!OfChXgczrIlpEZSFAv:decred.org/?anchor=$153125687135698lmQRO:decred.org

xaur avatar Aug 03 '18 21:08 xaur

Yes because of the limitations of the peeking APIs it cannot access old events without having to paginate to them which would be very expensive

t3chguy avatar Aug 03 '18 21:08 t3chguy

Does it mean a won't fix?

xaur avatar Aug 03 '18 23:08 xaur

It'd mean having to persist things locally (duplicating data) but it's not completely out of the question

t3chguy avatar Aug 04 '18 04:08 t3chguy

Thanks. I guess I shall be using matrix.to "dispatch" links then -- tried opening some "old" links in Riot and they open pretty fast.

xaur avatar Aug 04 '18 06:08 xaur

By the way, why old message links work in riot?

Jus tried this link from Jul 2 and it opened pretty fast. Does riot persist things locally (duplicating data) as you mentioned?

P.S. Apologies for novice questions, I'm trying to migrate my reports from linking to Slack messages to linking to Matrix messages and looking for most reader-friendly solutions. Expiring links and #60 are two obstacles I encountered

xaur avatar Aug 04 '18 09:08 xaur

Riot uses the context api which is relatively expensive and would have to be done for each event, matrix-static pulls pages of events in order and already runs into issues with hitting the server too much

t3chguy avatar Aug 04 '18 10:08 t3chguy

Related: #115

bkil avatar May 29 '23 16:05 bkil