Use try/catch around channel / lbry URLs in commentron data + notifications #7143

Closed
opened 2021-09-21 17:27:34 +02:00 by tzarebczan · 1 comment
tzarebczan commented 2021-09-21 17:27:34 +02:00 (Migrated from github.com)

There's times when commentron data may be incorrect, i.e. not have a proper channel name, missing channel name, etc. Right now the app crashes when this happens while parsing those URLs.

There's times when commentron data may be incorrect, i.e. not have a proper channel name, missing channel name, etc. Right now the app crashes when this happens while parsing those URLs.
tzarebczan commented 2021-09-30 23:41:34 +02:00 (Migrated from github.com)

Still crashing though - does it need to be handled differently?
image

Still crashing though - does it need to be handled differently? ![image](https://user-images.githubusercontent.com/8120721/135533921-17afa1d1-14dc-41d4-9105-69a141cf9847.png)
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-desktop#7143
No description provided.