Remove explicit locking in OpenIdConnectCachingSecurityTokenProvider

[keegan-caruso]

Fixes #3078

See #3124 and #3118 for more context.

The lock isn't needed if the update is done to the config object itself, since it is a reference type, it will be an atomic update. This removes the need to keep two separate fields in sync.

[keegan-caruso]

@mderriey or @alex-set would either of you want to take a look at this?

[github-actions]

Summary

Summary

Generated on:	1/16/2025 - 2:24:48 AM
Coverage date:	1/16/2025 - 2:23:42 AM - 1/16/2025 - 2:24:38 AM
Parser:	MultiReport (2x Cobertura)
Assemblies:	0
Classes:	0
Files:	0
Line coverage:
Covered lines:	0
Uncovered lines:	0
Coverable lines:	0
Total lines:	0
Covered branches:	0
Total branches:	0
Method coverage:	Feature is only available for sponsors

Coverage

No assemblies have been covered.

[jmprieur]

Super elegant!
Thanks @keegan-caruso

[jennyf19]

[github-actions]

Summary

Summary

Generated on:	1/16/2025 - 6:17:12 PM
Coverage date:	1/16/2025 - 6:16:13 PM - 1/16/2025 - 6:17:08 PM
Parser:	MultiReport (2x Cobertura)
Assemblies:	0
Classes:	0
Files:	0
Line coverage:
Covered lines:	0
Uncovered lines:	0
Coverable lines:	0
Total lines:	0
Covered branches:	0
Total branches:	0
Method coverage:	Feature is only available for sponsors

Coverage

No assemblies have been covered.

[jmprieur]

Tested locally

[alex-set]

@mderriey or @alex-set would either of you want to take a look at this?

Sorry, I know I'm late, but it looks great to me.