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.

Sign up for GitHub

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

Jump to bottom

Do not use os.Exit #9

Open
milabs opened this issue Mar 22, 2021 · 0 comments
Open

Do not use os.Exit #9

milabs opened this issue Mar 22, 2021 · 0 comments

Comments

@milabs
Copy link

milabs commented Mar 22, 2021

It's not possible to use that module as it uses os.Exit(...) on failure which causes application to exit even if coindesk logic run inside the go-routine. I'd recommend to use panic() instead as it could be handled with recover().
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@milabs