Skip to content

Attempt to fix Flask failing to reestablish connection with Cockroach after session closing

Callum Dickinson requested to merge db-disconnect-fix into master
  • Fix the situation where Hot Potato returns a 500 error on the first request after the connection between Hot Potato and CockroachDB is severed, caused by active connections not being checked for liveliness (and restarted) before they are used
  • Add a unit test (GitLab CI only unfortunately for the moment) for this situation
  • Get rid of the Hot Potato activity timer in the robot-test GitLab CI job, because it was broken (checked the wrong port number thus always waited the full minute) and appeared to no longer do anything anyway, based on how the test works now
Edited by Callum Dickinson

Merge request reports