make parseURI (URL) support all possible lbry urls #189

Closed
opened 2019-08-23 02:32:23 +02:00 by jessopb · 2 comments
jessopb commented 2019-08-23 02:32:23 +02:00 (Migrated from github.com)

Currently, parseURL errors on the following because it doesn't expect modifiers after /
'lbry://@channel#2/stream:5'
'lbry://@channel$2/stream#5'

Currently, parseURL errors on the following because it doesn't expect modifiers after / 'lbry://@channel#2/stream:5' 'lbry://@channel$2/stream#5'
tzarebczan commented 2019-09-26 16:57:21 +02:00 (Migrated from github.com)

@jessopb this was fixed right?

@jessopb this was fixed right?
jessopb commented 2019-10-18 17:22:21 +02:00 (Migrated from github.com)

It looks like it - though it looks like the app doesn't respect $2 and such.

It looks like it - though it looks like the app doesn't respect $2 and such.
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-redux#189
No description provided.