TaskResultCache incorrectly shares the same lock object for all the keys.

[Erarndt]

There is a very subtle bug in TaskResultCache that is causing the same lock object to be used rather than a different one per key. Here's the offending code:

object lockObject = _perTaskLock.GetOrAdd(key, static () => new object());

This erroneously caches the delegate that is used. The intent was to pass in a delegate that will create a new object and cache/lock on that.

[dotnet-policy-service]

@Erarndt Issue is missing Type label, remember to add a Type label

[jeffkl]

🤦

[jeffkl]

Fixed by NuGet/NuGet.Client#5808