authentication code should not depend on DBClient

The "authcommon" target has a dependency on DBClient because of the following method:

bool authenticateInternalUser(DBClientWithCommands* conn);

authenticateInternalUser() should become a method on the DBClientWithCommands class so DBClient depends on the auth code and not the other way around.