Make BuildInfoRecorder.java thread safe

jfrog / artifactory-maven-plugin

A Maven plugin to resolve artifacts from Artifactory, deploy artifacts to Artifactory, capture and publish build info.

https://www.jfrog.com/confluence/display/JFROG/Maven+Artifactory+Plugin

Apache License 2.0

24 stars 26 forks source link

Make BuildInfoRecorder.java thread safe #78

Closed rdeangelis83 closed 1 year ago

rdeangelis83 commented 1 year ago

The buildTimeDependencies collection is a SynchronizedCollection but like mentioned in the JavaDoc of Collections.synchrizedCollection() it is necessary to use a synchronize block by traversing the collection via iterator, what will be happen by hands over the collection to the addAll method.

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.

github-actions[bot] commented 1 year ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

rdeangelis83 commented 1 year ago

I have read the CLA Document and I hereby sign the CLA