Auditing plus fixes for terms-of-use acceptances

[labkey-adam]

Rationale

One client request and several issues:

• Request to add auditing of terms acceptance: https://github.com/LabKey/internal-issues/issues/1500
• In the login API action, pass the authenticated user into the setTerms* methods in addition to the ViewContext. The user isn't set on the ViewContext yet.
• Fix the login API action MFA case, where the auth response still has a null user
• When user impersonation is taking place and terms are accepted, we were setting the terms acceptance timestamp for the impersonator (correct), but always reading timestamps for the impersonated user (incorrect).

Related Pull Requests

• Configuration option to remember terms-of-use acceptance #7776