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

Stop panicking! #66

Closed
futurechimp opened this issue Jan 20, 2020 · 0 comments · Fixed by #101
Closed

Stop panicking! #66

futurechimp opened this issue Jan 20, 2020 · 0 comments · Fixed by #101
Assignees
@jstuczyn
Milestone
0.4.0

Comments

@futurechimp
Copy link
Contributor

futurechimp commented Jan 20, 2020 •
edited
Loading

We've been mildly sloppy as we rush to get things running. Sanitize all uses of unwrap and expect to ensure we don't accidentally destroy anything in situations where we expect bad inputs, handling errors gracefully.
@futurechimp futurechimp added this to the 0.4.0 milestone Jan 20, 2020
@jstuczyn jstuczyn mentioned this issue Jan 24, 2020
Merged
@jstuczyn jstuczyn self-assigned this Jan 24, 2020
@mmsinclair mmsinclair moved this to Done in Core systems May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jstuczyn jstuczyn

Labels
None yet
Projects
No open projects
Core systems
Archived in project
Milestone
0.4.0
Development

Successfully merging a pull request may close this issue.

Feature/panic improvements nymtech/nym
2 participants
@futurechimp @jstuczyn