commons-validator icon indicating copy to clipboard operation
commons-validator copied to clipboard

Remove commons-collections; Replace FastHashMap with ConcurrentHashMap

Open chrismaeda opened this issue 3 years ago • 13 comments

We needed to eliminate commons-collections 3 for a large corp. due to security concerns.

This replaces FastHashMap with ConcurrentHashMap and removes commons-collections.

FYI We also submitted a PR to commons-beanutils2 to modify that repo to be a commons-collections -free version that can replace beanutils 1.9.4.

chrismaeda avatar Mar 03 '21 04:03 chrismaeda

-1 due to binary compatibility. Also the POM changes are invalid.

garydgregory avatar Mar 03 '21 12:03 garydgregory

HI @chrismaeda You can check if the change brake the binary by running japicmp:cmp Regards,

arturobernalg avatar Mar 08 '21 18:03 arturobernalg