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

500 error on provisioning_csv endpoint when enrollment_term_id is a number #2419

Open
michaelPotter opened this issue Oct 11, 2024 · 0 comments

Comments

@michaelPotter
Copy link

michaelPotter commented Oct 11, 2024

Summary:

For the last two days we've been getting a 500 error in the call to:

/api/v1/accounts/${account}/reports/provisioning_csv
{"parameters": {"xlist": true, "enrollment_term_id": 4550}}

Steps to reproduce:

  1. Call the reports provisioning_csv api with enrollment_term_id set to a number. Note: from testing in curl, it works if enrollment_term_id is a string, e.g. "4550".

Expected behavior:

A successful response.

Actual behavior:

500 Exception code

Additional notes:

Based on the git tags, looks like there was a release on 2024-10-09 which aligns with when we started getting the error.

It also looks like there was one commit 4c5545e to app/controllers/account_report_controller.rb since the 2024-09-25 release that updated the validation on the enrollment_term_id field. I don't know ruby, but I bet this bit enrollment_term_id.split(",") is trying to call a string method on a number and crashing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant