Use ConcurrentBag instead of a List because the .Add method of List is not thread safe.

dotnet / sign

Code Signing CLI tool supporting Authenticode, NuGet, VSIX, and ClickOnce

MIT License

439 stars 84 forks source link

Use ConcurrentBag instead of a List because the .Add method of List is not thread safe. #715

Closed dlemstra closed 2 weeks ago

dlemstra commented 2 weeks ago

This PR fixes a mistake that I made in my earlier pull request. I did not take in account that List.Add is not thread safe. Using a ConcurrentBag instead fixes this issue.