URL encode error on share link : using non authorized characters ! And fail opening well formatted URL. #1052

Closed
opened 2020-11-10 14:26:32 +01:00 by oopen · 3 comments
oopen commented 2020-11-10 14:26:32 +01:00 (Migrated from github.com)

Share link not encoded

Steps to reproduce

  1. open a video on the android app
  2. click on share button
  3. the link is not url encoded, on fail to open on browser, sms app...

Expected behavior

The link should be URL encoded

Actual behavior

the link is not encoded and contain not allowed characters, like utf-8 icon, ! : ...

Not open well encoded links

Steps to reproduce

  1. open a encoded link with lbry app
  2. the content is not find, because the link is not decoded

Expected behavior

link decode, open content

Actual behavior

link not decoded, content not found

# Share link not encoded ### Steps to reproduce 1. open a video on the android app 2. click on share button 3. the link is not url encoded, on fail to open on browser, sms app... ### Expected behavior The link should be URL encoded ### Actual behavior the link is not encoded and contain not allowed characters, like utf-8 icon, ! : ... # Not open well encoded links ### Steps to reproduce 1. open a encoded link with lbry app 2. the content is not find, because the link is not decoded ### Expected behavior link decode, open content ### Actual behavior link not decoded, content not found
kekkyojin commented 2020-12-15 18:30:28 +01:00 (Migrated from github.com)

Is it happening on any video or on a few of them?

Is it happening on any video or on a few of them?
tzarebczan commented 2020-12-15 20:03:54 +01:00 (Migrated from github.com)

Can you give us an example of a URL that's happening on? We may need to encode the claim name on the share portion. We just added that on the lbry.tv/odysee side.

Can you give us an example of a URL that's happening on? We may need to encode the claim name on the share portion. We just added that on the lbry.tv/odysee side.
oopen commented 2021-01-07 11:40:00 +01:00 (Migrated from github.com)

On android app: I click Share button, like generated :

https://lbry.tv/La-Peur,-Nos-Attentats,-c'est-VOTRE-Sécurité!-Les-Guignols#6

As you can see, the link contain not url allowed caraters like : , !, é '
there characters must be urlencoded.

Secondly, if I urlencode the link to :

https://lbry.tv/La-Peur%2C-Nos-Attentats%2C-c%27est-VOTRE-Se%CC%81curite%CC%81%21-Les-Guignols#6

This link can't be open by android app,
but mobiles & desktop browsers can.

So link is not urldecoded on android app.

This bug appear on all videos with not allowed characters (many).

It cause problems to share the link, because other apps can't parse the link correctly and break on first not allowed characters.

https://tools.ietf.org/html/rfc8820

On android app: I click Share button, like generated : [https://lbry.tv/La-Peur,-Nos-Attentats,-c'est-VOTRE-Sécurité!-Les-Guignols#6](https://lbry.tv/La-Peur,-Nos-Attentats,-c'est-VOTRE-Sécurité!-Les-Guignols#6) As you can see, the link contain not url allowed caraters like : `,` `!`, `é` `'` there characters must be urlencoded. Secondly, if I urlencode the link to : [https://lbry.tv/La-Peur%2C-Nos-Attentats%2C-c%27est-VOTRE-Se%CC%81curite%CC%81%21-Les-Guignols#6](https://lbry.tv/La-Peur%2C-Nos-Attentats%2C-c%27est-VOTRE-Se%CC%81curite%CC%81%21-Les-Guignols#6) This link can't be open by android app, but mobiles & desktop browsers can. So link is not urldecoded on android app. This bug appear on all videos with not allowed characters (many). It cause problems to share the link, because other apps can't parse the link correctly and break on first not allowed characters. [https://tools.ietf.org/html/rfc8820](https://tools.ietf.org/html/rfc8820)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
LBRYCommunity/lbry-android#1052
No description provided.