Clean up native auth common logging

[Yuki-YuXin]

Goal:

• Review our log statements and reduce to only those that are relevant and useful. Especially the logMethodCall should be much reduced. Not every method call is relevant to know about when debugging.
• Double check that the existing logs use string interpolation correctly to avoid leaking sensitive information in the logs.

Changes summary:

• Every log item has correlation argument. (make sure use getCorrelationId() )
• The NativeAuthMsalController.kt layer acts as the main log layer. Instead, remove logSessions under OAuth2Configuration, OAuth2Strategy, RequestProvider since the users care more about the outcome of the request than the construction and sending of the request. Apply the rule "only logger unknown error" to NativeAuthMsalController.
• Remove // TODO
• Add request and result Logger.infoWithObject in interactors.
• infoWithObject(final String tag, final String correlationID, final String message, final ILoggable object) like warnWithObject

Company PR:

Previous comments for reference: #2374
msal: AzureAD/microsoft-authentication-library-for-android#2071

[SammyO]

Why are we putting this back?

[SammyO]

Right, because there's a TODO there. IMO we don't need this log. Does the browser-delegated flow log something like this?

[Yuki-YuXin]

The browser-delegated flow log something like this:

In terms of the TokenResponse of MSAL:
MicrosoftStsOAuth2Strategy:TokenResult:MicrosoftStsTokenResponse INFO {"ext_expires_in":4165,"expires_in":4165,"mResponseReceivedTime":0,"scope":"email openid profile Application.Read.All User.Read","token_type":"Bearer"}

BaseController:TokenResult:MicrosoftStsTokenResponse INFO {"ext_expires_in":4165,"mRefreshTokenAge":"","mSpeRing":"","expires_in":4165,"mResponseReceivedTime":0,"scope":"email openid profile Application.Read.All User.Read","token_type":"Bearer"}

Thus update the contents of the log to include what MSAL displays. But we can remove this log as well because token seems very risky and sensitive to me.

[SammyO]

Yeah, let's remove this for now until we have a reason to put it back.

[GBakolas]

Nit: In the ApiResponse files where you have removed the calls to logMethodCall() there are the LogSession import statements left. These statements are unused now so they can be removed for code cleanup