Refactor api database code

[ConnorBP]

The current database code opens a new connection instance in each helper, this results in constant closes and opens. We instead want to pass connections by parameter in each helper function, and let the api route that calls it manage the opening and closing of the connection. Also look for any connection leaks. If there is a return statement it must close the database first.