Add warning when opening files externally #1761

Closed
opened 2018-07-12 06:36:26 +02:00 by tzarebczan · 2 comments
tzarebczan commented 2018-07-12 06:36:26 +02:00 (Migrated from github.com)

The Issue

When using the open button (and possibly when clicking the file location at the bottom of a downloaded claim), we should warn users that they are opening at their own risk. FIles played inside the app are sandboxed, but anything opened externally could be a risk. Maybe we only do this for non-playable/viewable types?

It would be similar to the modal when clicking a hyperlink in a claim.

System Configuration

  • LBRY Daemon version:
  • LBRY App version:
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. - App releases: https://github.com/lbryio/lbry-app/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on live chat at https://chat.lbry.io --> ## The Issue When using the open button (and possibly when clicking the file location at the bottom of a downloaded claim), we should warn users that they are opening at their own risk. FIles played inside the app are sandboxed, but anything opened externally could be a risk. Maybe we only do this for non-playable/viewable types? It would be similar to the modal when clicking a hyperlink in a claim. ## System Configuration <!-- For the app, this info is in the About section at the bottom of the Help page. You can include a screenshot instead of typing it out --> <!-- For the daemon, run: curl 'http://localhost:5279' --data '{"method":"version"}' and include the full output --> - LBRY Daemon version: - LBRY App version: - LBRY Installation ID: - Operating system: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here -->
neb-b commented 2018-07-16 15:15:09 +02:00 (Migrated from github.com)

This should use the same styling as the openExternalLink modal. Maybe we can combine them into one modal?

@btzr-io This might be a good one for you if you are interested.

This should use the same styling as the `openExternalLink` modal. Maybe we can combine them into one modal? @btzr-io This might be a good one for you if you are interested.
tzarebczan commented 2019-05-21 16:45:35 +02:00 (Migrated from github.com)
This was completed in https://github.com/lbryio/lbry-desktop/pull/2478
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#1761
No description provided.