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

V3 events (with slashes in them) break generated permalinks and cause you to lose your place in the timeline when forwarding events. #9520

Closed
unclechu opened this issue Apr 17, 2019 · 11 comments
Assignees

Comments

@unclechu
Copy link

Forwarding many events from one room to another is messed up in Riot.

  1. I can't just forward few events (in my case photos), like selecting few of them and forward them at once so I have to do it one by one, I'm feeling sick when I do it for 20 events for instance;

  2. Very strange bug I tried to avoid but failed. When you forward an event from room A to room B you just go back to room A to forward next event in the list, but when you press forward on next event scroll falls down to the last event in the room, so now you're lost the last item you were trying to forward, so after a forward it to B you now have to go to room A and find that last forwarded event, so hard to not get tired;

  3. As a possible way to avoid bug in pt. 2 I just imagined I could anchor on specific events. So what I was trying to do is clicking on current event I'm forwarding at the moment (clicking on time of event, on the left side) so I would have exact URI link in my browser history, forward an event and then just press "back" button in my browser, but it works only for one event, or sometimes somehow for few but always failing with error on screenshot:

Screenshot from 2019-04-17 23-14-16

Version information

  • Platform: web (in-browser)
  • Browser: Firefox Quantum 66.0.2 (64-bit)
  • OS: Fedora Workstation 28 (Twenty Eight)
  • URL: https://riot.im/app/
matrix-react-sdk version: <local>
riot-web version: 1.0.7
olm version: 3.0.0
@unclechu
Copy link
Author

I realized that events which have such issues are have slashes in their ids.
Like this one: $3/*****************/*********************** (* stands for [a-zA-Z0-9]).
Everything is okay with events which don't have slashes, like this: $*******************************************

@unclechu
Copy link
Author

It's probably related to parsing pathname by router.
And when I see that error on screenshot, I see only this:
https://riot.im/app/#/room/!******************:matrix.org/$3
So event id is sliced after first slash. When I just click on time of events it becomes:

https://riot.im/app/#/room/!******************:matrix.org/$3/*****************/***********************

But when I move back and forth in history it resolves to that error and link is sliced.

And if I follow by that full link in new tab, URI becomes sliced and I see this error:
Screenshot from 2019-04-18 01-13-10
If I try to join I see this:
Screenshot from 2019-04-18 01-13-26

@unclechu
Copy link
Author

unclechu commented Apr 17, 2019

I also reproduced that in events which have + symbol in their ids:
$***************************************+
In this case whole event id sliced from URI.

UPD: But it seems only when + at the end of id, this works okay:
$*********+***********************+*********

@lampholder lampholder changed the title Scroll is falling down after second event forwaring V3 events (with slashes in them) break generated permalinks and cause you to lose your place in the timeline when forwarding events. Apr 30, 2019
@lampholder
Copy link
Member

Thanks for the investigation @unclechu - you're right, these issues do seem to be cause on events with slashes (or perhaps +es) in them. I've updated the title to reflect that these are 'v3' events (my understanding is that event ids won't always be allowed to have slashes in them, but we have to fix riot to defend against them for current events.

@turt2live
Copy link
Member

matrix-org/matrix-react-sdk#2827 should have fixed the general case of this, which is currently unreleased (in 1.1.0-rc.1). The other portion of this is highly related to https://github.com/vector-im/riot-web/issues/9424 / #9149

@turt2live turt2live self-assigned this May 16, 2019
@turt2live
Copy link
Member

I think this is fixed by matrix-org/matrix-react-sdk#2827 - I can't reproduce it anymore.

@unclechu
Copy link
Author

@turt2live When this fix is supposed to be delivered to stable Riot branch? Will this issue be marked with some tag then?

@turt2live
Copy link
Member

It should have landed in 1.1.0

@unclechu
Copy link
Author

@turt2live Hmm... It seems it's already fixed in stable Riot branch (I mean https://riot.im/app/ by that).

@turt2live
Copy link
Member

Yes, that is 1.1.2 which includes 1.1.0

@unclechu
Copy link
Author

@turt2live Cool, thanks for fixing it!

su-ex added a commit to SchildiChat/element-web that referenced this issue Nov 25, 2022
* Make clear notifications work with threads ([\element-hq#9575](matrix-org/matrix-react-sdk#9575)). Fixes element-hq#23751.
* Change "None" to "Off" in notification options ([\element-hq#9539](matrix-org/matrix-react-sdk#9539)). Contributed by @Arnei.
* Advanced audio processing settings ([\element-hq#8759](matrix-org/matrix-react-sdk#8759)). Fixes element-hq#6278. Contributed by @MrAnno.
* Add way to create a user notice via config.json ([\element-hq#9559](matrix-org/matrix-react-sdk#9559)).
* Improve design of the rich text editor ([\element-hq#9533](matrix-org/matrix-react-sdk#9533)). Contributed by @florianduros.
* Enable user to zoom beyond image size ([\element-hq#5949](matrix-org/matrix-react-sdk#5949)). Contributed by @jaiwanth-v.
* Fix: Move "Leave Space" option to the bottom of space context menu ([\element-hq#9535](matrix-org/matrix-react-sdk#9535)). Contributed by @hanadi92.
* Make build scripts work on NixOS ([\element-hq#23740](element-hq#23740)).
* Fix integration manager `get_open_id_token` action and add E2E tests ([\element-hq#9520](matrix-org/matrix-react-sdk#9520)).
* Fix links being mangled by markdown processing ([\element-hq#9570](matrix-org/matrix-react-sdk#9570)). Fixes element-hq#23743.
* Fix: inline links selecting radio button ([\element-hq#9543](matrix-org/matrix-react-sdk#9543)). Contributed by @hanadi92.
* Fix wrong error message in registration when phone number threepid is in use. ([\element-hq#9571](matrix-org/matrix-react-sdk#9571)). Contributed by @bagvand.
* Fix missing avatar for show current profiles ([\element-hq#9563](matrix-org/matrix-react-sdk#9563)). Fixes element-hq#23733.
* Fix read receipts trickling down correctly ([\element-hq#9567](matrix-org/matrix-react-sdk#9567)). Fixes element-hq#23746.
* Resilience fix for homeserver without thread notification support ([\element-hq#9565](matrix-org/matrix-react-sdk#9565)).
* Don't switch to the home page needlessly after leaving a room ([\element-hq#9477](matrix-org/matrix-react-sdk#9477)).
* Differentiate download and decryption errors when showing images ([\element-hq#9562](matrix-org/matrix-react-sdk#9562)). Fixes element-hq#3892.
* Close context menu when a modal is opened to prevent user getting stuck ([\element-hq#9560](matrix-org/matrix-react-sdk#9560)). Fixes element-hq#15610 and element-hq#10781.
* Fix TimelineReset handling when no room associated ([\element-hq#9553](matrix-org/matrix-react-sdk#9553)).
* Always use current profile on thread events ([\#9524](matrix-org/matrix-react-sdk#9524)). Fixes element-hq#23648.
* Fix `ThreadView` tests not using thread flag ([\element-hq#9547](matrix-org/matrix-react-sdk#9547)). Contributed by @MadLittleMods.
* Handle deletion of `m.call` events ([\element-hq#9540](matrix-org/matrix-react-sdk#9540)). Fixes element-hq#23663.
* Fix incorrect notification count after leaving a room with notifications ([\element-hq#9518](matrix-org/matrix-react-sdk#9518)). Contributed by @Arnei.
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

3 participants