lz233 / Tarnhelm

The magic to clean sharing links up.
https://tarnhelm.project.ac.cn
GNU General Public License v3.0
422 stars 17 forks source link

Incorrect parsing of the Telegram invitation link #24

Closed omg-xtao closed 1 year ago

omg-xtao commented 2 years ago

Example: https://t.me/+ABC Copied as: https://t.me/ ABC

https://github.com/lz233/Tarnhelm/blob/8825c68e5af9484c8c63287ee44a522cc9b69d7b/app/src/main/java/cn/ac/lz233/tarnhelm/util/ktx/String.kt#L66

Temporary remedies: eyJhIjoidGciLCJiIjpbInQubWVcLyAiXSwiYyI6WyJ0Lm1lXC8rIl0sImQiOiJvbWcteHRhbyJ9

lz233 commented 2 years ago

Because some apps will directly connect URLs and characters without spacing, like https://xxxxxxx(来自@xxxx), so we must do decodeURL() since okhttp will automatically encode it. If ignoring the irregularity, we can remove this behavior.

lz233 commented 2 years ago

So I have no idea at this time(

omg-xtao commented 2 years ago

~

lz233 commented 2 years ago

It is an obvious problem and should not be closed

Mosney commented 1 year ago

原来是Tarnhelm搞的鬼,我还以为是Gboard的分词删掉的

lz233 commented 1 year ago

Fixed in 1.5.4 and will be published in Google Play later. Please tell me if the problem still occur.