Use a new MariaDB connection object for every api request

[rocketeerbkw]

Checklist

• Affected Issues have been mentioned in the Closing issues section
• Documentation has been written/updated.
• Changelog entry has been written

Moved the sqlClient constructor inside of the Apollo server and express middlewares so that a new connection will be generated for every request. Putting it in the Apollo Server and passing it to resolvers via context means the connection will be shared when multiple resolvers need to be called to fulfill one request.

90% of the code changes are just adding sqlClient to the resolvers and auto-formatting changes. The magic happens in apolloServer.js and authMiddleware.js

Changelog Entry

Improvement - Each API request will generate a new MariaDB connection (#920)

Closing issues

closes #920

[rocketeerbkw]

Seeing a lot of error messages now about too many connections:

api_1 | error: message=Too many connections, stack=Error: Too many connections, code=1040

It looks like connections must explicitly be closed, combined with a new connection for every request makes sense that it runs out. Just looking at the docs for apollo server I can't see any way to close the connection after a request has been "finished." Will have to dig into code or ask SO or somewhere.

[rocketeerbkw]

Found an undocumented plugin system in apollo server that can listen to request pipeline events. I no longer get connection errors, and tests are green.