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

IssueHandle::delete_label panics #456

Closed
chorman0773 opened this issue Sep 5, 2023 · 5 comments
Closed

IssueHandle::delete_label panics #456

chorman0773 opened this issue Sep 5, 2023 · 5 comments
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@chorman0773
Copy link

(Code is using octocrab = "0.30.1")
When calling IssueHandle::delete_label, IssueHandle panics. This appears to be because it expects a response in Json format, but according to github's documentation, the endpoint does not return any data (on success, it yields 204 No Content).

@chorman0773
Copy link
Author

Correction: It is returning an error, rather than panicking (the backtrace was throwing me off). However, the root cause of the issue remains: it's trying to parse a response that the API does not provide.

@XAMPPRocky
Copy link
Owner

Thank you for your issue! I don't have time to fix this myself right now, but the solution is simple, we just need use _delete instead of delete which doesn't try any of the parsing of the request.

@XAMPPRocky XAMPPRocky added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers labels Sep 5, 2023
@chorman0773
Copy link
Author

chorman0773 commented Sep 5, 2023 via email

@manchicken
Copy link
Contributor

I think the PR I just submitted yesterday, #503 should resolve this.

@XAMPPRocky
Copy link
Owner

Thank you for your issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants