Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

user_ids aren't escaped when generating urls [sending invites, joining rooms, etc] for federation use. #3019

Closed
walle303 opened this issue Mar 21, 2018 · 1 comment
Labels
z-bug (Deprecated Label) Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution z-p2 (Deprecated Label)

Comments

@walle303
Copy link

As / is an allowed character in user ids, they need to be escaped when generating urls, because right now if you have a / in your username and try to join a room or send an invite over federation, synapse generates urls like .../@user/id/with/forward/slashes/in/it:domain.
These, for obvious reasons, do not work. Instead they should be properly escaped, so .../%40user%id%2Fwith%2Fforward%2Fslashes%2Fin%2Fit%3Adomain would be used instead, or, at least escape localparts, so .../@user%2Fid%2Fwith%2Fforward%2Fslashes%2Fin%2Fit:domain instead.

@neilisfragile neilisfragile added Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution z-p2 (Deprecated Label) z-bug (Deprecated Label) labels Mar 27, 2018
@richvdh
Copy link
Member

richvdh commented Apr 10, 2018

fixed by #3082

@ara4n ara4n closed this as completed Apr 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
z-bug (Deprecated Label) Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

4 participants