Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve error message from ironoxide init timeout #275

Open
BobWall23 opened this issue Jun 29, 2022 · 0 comments
Open

Improve error message from ironoxide init timeout #275

BobWall23 opened this issue Jun 29, 2022 · 0 comments

Comments

@BobWall23
Copy link
Member

@cjyar commented on Thu Jun 16 2022

As a naive user, who may not remember how the TSP works, I want a clear error message that tells me what the problem is if ironoxde can't be initialized.

The current error message is:

thread 'main' panicked at 'Startup failed: failed to initialize the IronCore SDK with the provided credentials. Operation 'InitializeSdk' timed out after 30000ms', src/util/serde.rs:13:19

AC:

  • If the cause of the error comes from a network connection, the error message should say what it was trying to connect to, and give as much information about why it failed as we have.

@BobWall23 commented on Wed Jun 29 2022

If we added a timeout on the request that was shorter than 30 sec, we would get the error from the reqwest client instead of the generic timeout error, and could propagate that up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant