Audit all info and verbose logs and use lambdas to generate strings

[bgavrilMS]

Fixes #3901

Changes proposed in this request

• add logging methods that use lambdas, which do not execute unless the log level is met (this is inspired by high-performance logging from https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-7.0)
• for verbose logging, i removed the overloads with strings. Use only Func
• for info it's a mix bag - strings and Func, since info is ON by default and the Func introduces an extra check.

Testing
unit

Performance impact

Method	Mean	Error	StdDev	Median	Min	Max	Gen0	Allocated
'Compute and log expensive string via direct string'	1,292.38 ns	35.423 ns	33.135 ns	1,287.09 ns	1,253.46 ns	1,371.78 ns	0.1488	944 B
'Compute and log expensive string via Func'	1,369.27 ns	41.625 ns	34.759 ns	1,362.51 ns	1,296.23 ns	1,436.84 ns	0.1678	1056 B
'Compute and skip logging expensive string'	628.37 ns	10.844 ns	10.143 ns	626.69 ns	611.36 ns	650.44 ns	0.0591	376 B
'No compute (Func) and skip logging expensive string'	14.14 ns	0.286 ns	0.239 ns	14.14 ns	13.72 ns	14.66 ns	0.0153	96 B

** AcquireTokenForClient perf tests **

These were modified to run with WithLogging(Console.WriteLine(msg), LogLevel.Info)

Method	CacheSize	EnableCacheSerialization	Mean	Min	Max	Gen0	Gen1	Gen2	Allocated
AcquireTokenForClient	(1, 10)	False	3.114 ms	3.057 ms	3.204 ms	7.8125	-	-	51.26 KB
AcquireTokenForClient	(1, 10)	True	3.204 ms	3.163 ms	3.280 ms	46.8750	15.6250	-	287.65 KB
AcquireTokenForClient	(1, 1000)	False	3.121 ms	3.059 ms	3.160 ms	27.3438	-	-	190.96 KB
AcquireTokenForClient	(1, 1000)	True	30.336 ms	29.677 ms	31.857 ms	2687.5000	1312.5000	625.0000	17010.89 KB
AcquireTokenForClient	(10000, 10)	False	3.116 ms	3.030 ms	3.184 ms	7.8125	-	-	51.28 KB
AcquireTokenForClient	(10000, 10)	True	4.117 ms	3.383 ms	4.852 ms	46.8750	15.6250	-	287.65 KB

Method	CacheSize	EnableCacheSerialization	Mean	Min	Max	Gen0	Gen1	Gen2	Allocated
AcquireTokenForClient	(1, 10)	False	2.990 ms	2.907 ms	3.063 ms	7.8125	-	-	50.23 KB
AcquireTokenForClient	(1, 10)	True	3.092 ms	3.024 ms	3.180 ms	42.9688	11.7188	-	287.06 KB
AcquireTokenForClient	(1, 1000)	False	3.012 ms	2.932 ms	3.064 ms	23.4375	-	-	158.98 KB
AcquireTokenForClient	(1, 1000)	True	29.365 ms	27.786 ms	31.487 ms	2625.0000	1250.0000	625.0000	17065.87 KB
AcquireTokenForClient	(10000, 10)	False	2.966 ms	2.916 ms	3.019 ms	7.8125	-	-	50.24 KB
AcquireTokenForClient	(10000, 10)	True	3.080 ms	3.010 ms	3.147 ms	42.9688	11.7188	-	287.03 KB

Documentation
n/a

[pmaytak]

Cool stuff. Based on perf tests, seems like it does help to not compute when logging is disabled. Would be nice though if there was a syntactic sugar to not add () => part - parameter is Func, you pass in a string, but compiler figures out to change it to a Func.