readium / r2-testapp-swift

BSD 3-Clause "New" or "Revised" License
146 stars 38 forks source link

LCP: missing warning if importing a license but 404 content link #235

Closed llemeurfr closed 5 years ago

llemeurfr commented 5 years ago

Let's take a license from a local + inactive LCP server, import it in the test app. The import fails because the content is not reachable through the frontend server. But the user sees no message; it should. The error message can be "Import failed because remote content is unreachable".

mickael-menu-mantano commented 5 years ago

Actually there's an error but only after a timeout of ~60 seconds. This seems to only occur when using a local address: I tried using a random domain name instead of 192.168... in the license, and it failed immediately.