f910381 breaks functions that establish their own db connections

[eric-hunt]

Moving connection handling (withr) to the db_query function breaks other functions (see [hRvstAPI::update_db()] for a good example) which establish their own database connection for repeated use. The db_query function should be updated with logic where if an active connection is provided, the local path argument is ignored (see eric-hunt/htce@6a63d96 for example of how to do this).