SynchronossPartGenerator should reuse PartBodyStreamStorageFactory [SPR-16727]

[spring-projects-issues]

Julien Hoarau opened SPR-16727 and commented

SynchronossPartGenerator creates a new NioMultipartParser with:

NioMultipartParser parser = Multipart.multipart(context).forNIO(listener);

This will create a new DefaultPartBodyStreamStorageFactory each time. DefaultPartBodyStreamStorageFactory constructor will call File.exists, this could be "slow" on some system. We should have a single instance of DefaultPartBodyStreamStorageFactory and reuse it.

---

Issue Links:

• Race-Condition in Multipart upload proxy scenario [SPR-16639] #21180 Race-Condition in Multipart upload proxy scenario

[spring-projects-issues]

Rossen Stoyanchev commented

This could also help to address the race condition from #21180.