mitodl / rapid-response-xblock

a django app plug-in for edx-platform
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

IndexError: list index out of range #119

Closed sentry-io[bot] closed 2 years ago

sentry-io[bot] commented 2 years ago

Sentry Issue: OPENEDX-RESIDENTIAL-1J8

IndexError: list index out of range
  File "eventtracking/backends/routing.py", line 133, in send_to_backends
    backend.send(event)
  File "rapid_response_xblock/logger.py", line 82, in send
    sub = self.parse_submission_event(event)
  File "rapid_response_xblock/logger.py", line 60, in parse_submission_event
    submission_key, submission = list(event_submissions.items())[0]

Unable to send event to backend: rapid_response
arslanashraf7 commented 2 years ago

While debugging the issue it turns out that we already have fixed it in an earlier version of Rapid Response xBlock some time ago.

The line of code that this error refers to has been fixed in an earlier PR https://github.com/mitodl/rapid-response-xblock/pull/93 to only send our events in case there is absolutely one submission and the fix was released in Rapid Response xBlock - 0.1.0.

Looking at the sentry log I see we are still using version 0.0.9 and installing Rapid Response version 0.1.0 or above should fix this error.

Attaching some ref screenshots below:

Screenshot 2021-11-12 at 5 46 09 PM Screenshot 2021-11-12 at 5 53 23 PM

CC: @pdpinch

pdpinch commented 2 years ago

I opened an issue in salt-ops. I'm not sure exactly how we specify the updated version though.

pdpinch commented 2 years ago

https://github.com/mitodl/salt-ops/issues/1485

pdpinch commented 2 years ago

Unless we hear this is urgent from the residential MITx team, we'll wait to pick up this fix with the maple release.

pdpinch commented 2 years ago

This was fixed for maple in https://github.com/mitodl/ol-infrastructure/commit/a33dff0c39985249205d861c4028792b405f3ff7