Can't open app on Mac/Linux if GitHub API call fails #1774

Closed
opened 2018-07-16 20:39:49 +02:00 by tzarebczan · 1 comment
tzarebczan commented 2018-07-16 20:39:49 +02:00 (Migrated from github.com)

The Issue

GitHub was down for a few minutes and someone reported not being able to access the app. Add additional error checking for the GitHub API call (e1210f7805/src/main/index.js (L184)) so that it doesn't prevent app startup if GitHub is down.

There is an uncaught Json error called on startup for Mac/Linux or on the help page for Windows. (don't have a screenshot). Probably occurring here: e1210f7805/src/main/index.js (L197)

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-desktop/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on live chat at https://chat.lbry.io --> ## The Issue GitHub was down for a few minutes and someone reported not being able to access the app. Add additional error checking for the GitHub API call (https://github.com/lbryio/lbry-desktop/blob/e1210f78051e984f26905c01f5d7eac454adad01/src/main/index.js#L184) so that it doesn't prevent app startup if GitHub is down. There is an uncaught Json error called on startup for Mac/Linux or on the help page for Windows. (don't have a screenshot). Probably occurring here: https://github.com/lbryio/lbry-desktop/blob/e1210f78051e984f26905c01f5d7eac454adad01/src/main/index.js#L197 ## 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 -->
tzarebczan commented 2019-03-16 23:28:15 +01:00 (Migrated from github.com)

This was fixed in 6126eb1bf4

This was fixed in https://github.com/lbryio/lbry-desktop/commit/6126eb1bf4248206aff119f2ec2f2a72d0f63ea7
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#1774
No description provided.