search

instructure / canvas-lms

The open LMS by Instructure, Inc.
https://github.com/instructure/canvas-lms/wiki
GNU Affero General Public License v3.0
5.53k stars 2.46k forks source link

API users/:user/courses fails when attempting to include deleted enrollments #1332

Open fuzzy76 opened 6 years ago

fuzzy76 commented 6 years ago

Set up:

  1. Have a user with deleted enrollments
  2. Do a GET to users/:user/courses?include[]=term

Result: 500 internal server error

fuzzy76 commented 6 years ago

Come to think of it, this might not be the optimal place for this report. I won't be offended if you delete it. :)

simonista commented 6 years ago

Can you view /error_reports and post the error stacktrace?

fuzzy76 commented 6 years ago

That's the problem. This is an instructure-hosted instance, and I don't seem to have access to those. So I should probably have taken this through those channels?

simonista commented 6 years ago

Yeah, if you can submit a ticket through the help menu in the UI, support should be able to help. Keep us posted, 500 can be indicative of a code error.

fuzzy76 commented 6 years ago

Correction.... include[]=term works fine while include[0]=term (which is what PHP creates when given arrays as query parameters) doesn't...

stale[bot] commented 4 years ago

Thanks for contributing to this issue. As it has been 2 years since the last activity, we are automatically closing the issue in 30 days. This is often because the request was already solved in some way and it just wasn't updated or it's no longer applicable. If that's not the case, please respond before the issue is closed, or post a message on the mailing list. We'll gladly take a look again!

fuzzy76 commented 4 years ago

This issue is still there... As described above.