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

[BUG] Prevent reqwest from panic-ing when user does not exist #25

Closed
ANF-Studios opened this issue Feb 11, 2021 · 1 comment
Closed

[BUG] Prevent reqwest from panic-ing when user does not exist #25

ANF-Studios opened this issue Feb 11, 2021 · 1 comment
Assignees
@ANF-Studios
Labels
bug Something isn't working

Comments

@ANF-Studios
Copy link
Member

Describe the bug
reqwest panics in the user_get method if the user is not found and nothing is returned by the API (other than the status message)

To Reproduce

  1. Get a user that you're sure does not exist
  2. Run the code
  3. Code panics

Expected behavior
N/A
@ANF-Studios ANF-Studios added the bug Something isn't working label Feb 11, 2021
@ANF-Studios ANF-Studios self-assigned this Feb 11, 2021
ANF-Studios added a commit that referenced this issue Feb 11, 2021
@ANF-Studios
Safer if-else checks ( #25 )
ef01256
@ANF-Studios
Copy link
Member Author

Fixed with commit master/ef01256dba00f8b0c18660c6d056ef410206d3ca

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ANF-Studios ANF-Studios

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ANF-Studios