telegramdesktop / tdesktop

Telegram Desktop messaging app
https://desktop.telegram.org/
Other
26.13k stars 5.18k forks source link

Linking more URL schemes #28364

Closed Dietr1ch closed 1 month ago

Dietr1ch commented 1 month ago

Is your feature request related to a problem?

Protocols other than http/https are not linked, like ftp://user:password@host:port/path, steam://launch/730 or todo://new/task?=buyMilk

Describe the solution you'd like

Support linking custom URL schemes is added, and text like ftp://user:password@host:port/path or steam://launch/730 becomes a link on sent messages.

IANA has a list of URL schemes, but today many apps use their own custom ones.

Describe alternatives you've considered

Not adding support, or using a whitelist. Having a whitelist might seem like a security feature at first, but if http:// links to fake and shady things are already allowed, then I don't think it's a real concern.

Additional context

ftp:// and steam:// are not linked, image

Lists of URI schemes

ilya-fedin commented 1 month ago

Link detection is server side. Clients just respect entities

Dietr1ch commented 1 month ago

Thanks for triaging it.

I found this on the API side: https://bugs.telegram.org/c/41466