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

Expose cause of initialization errors #6

Open
ramosbugs opened this issue Feb 15, 2022 · 4 comments
Open

Expose cause of initialization errors #6

ramosbugs opened this issue Feb 15, 2022 · 4 comments

Comments

@ramosbugs
Copy link

Is your feature request related to a problem? Please describe.

The Client provides an initialized_async method to wait until it has completed initialization. The function returns false if an error occurs, but there doesn't seem to be a way to access the error for logging purposes. This could make it a challenge to debug initialization issues.

Describe the solution you'd like

A more idiomatic interface in Rust would be to have initialized_async return something like Result<(), InitError> and define an InitError type to describe the different failure conditions that might arise.

Describe alternatives you've considered

A separate method could be used to retrieve the error (e.g., returning Option<InitError>.

@keelerm84
Copy link
Member

Thank you for the feedback! I have filed this request internally as sc-142483.

keelerm84 pushed a commit that referenced this issue Nov 7, 2022
…-use-extracted-rust-server-sdk-evaluation

[ch102632] Use extracted rust server sdk evaluation crate
@aalexandrov
Copy link

👍 for this.

@gabriel-lima
Copy link

+1 for this, I am having an unknown issue which for some reason client is not initializing.

@saks
Copy link

saks commented Jun 15, 2023

Experiencing the same issue, we had to spend quite a bit of time debugging and only found an issue once enabled TRACE logs. Would be great to have an explicit error returned when SDK key is wrong, or any other error happened (like timeout, etc).

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

5 participants