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

Fix failed submission status request #82

Merged
merged 1 commit into from
Sep 16, 2024
Merged

Conversation

avborup
Copy link
Owner

@avborup avborup commented Sep 16, 2024

Kattis changed their API to return different fields. Specifically, they removed fields from the response that we expected to be present, leading to an error.

Luckily, we do not actually use those missing fields, so no logic has to be changed.

The error was:

Submitted solution to https://open.kattis.com/submissions/....

Error: Failed to get submission status from Kattis

Run with --verbose for more information

And the root cause is pretty obvious from the verbose output:

Submitted solution to https://open.kattis.com/submissions/....

Error:
   0: Failed to get submission status from Kattis
   1: Failed to read submission status response from Kattis
   2: error decoding response body: missing field `testdata_groups_html` at line 11 column 661
   3: missing field `testdata_groups_html` at line 11 column 661

Location:
   .../kitty/src/commands/submit.rs:266

@avborup avborup added the bug Something isn't working label Sep 16, 2024
@avborup avborup self-assigned this Sep 16, 2024
@avborup avborup merged commit 985703a into master Sep 16, 2024
3 checks passed
@avborup avborup deleted the fix/submission-error branch September 16, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant