Skip to content
Commit 2f8b557a authored by Wadeck Follonier's avatar Wadeck Follonier Committed by Oleg Nenashev
Browse files

[JENKINS-55050] Reduce performance impact of SEC-904 (esp. zip) (#3861)



* [JENKINS-55050] Reduce log impact on performance
- especially in case of large / deep folder that is fully illegal
- could reduce time by a factor of around 2.

* Use SystemProperties

* [JENKINS-55050] Reduce performance impact of the security release
- SEC-904 increased the time taken to zip files from
DirectoryBrowserSupport views (workspace / artifact)
- This PR proposes a different approach for some of the highly impact scenario

* Update core/src/main/java/jenkins/util/VirtualFile.java

Co-Authored-By: default avatarWadeck <Wadeck@users.noreply.github.com>

* Return in case of interruption instead of continuing
- The interruption status flag is not really respected otherwise

* Update core/src/main/java/jenkins/util/VirtualFile.java

Co-Authored-By: default avatarWadeck <Wadeck@users.noreply.github.com>

* Improve ZipEntry writing
- Add logging on FileSystemException

* Minor refactoring in DirScanner

* Correct conjugation, size for arraylist and dead code removal

* Adjust the PR using the feedbacks

* Adjust the PR using the feedbacks (part 2)

* Adjust the PR using the feedbacks (part 3)
parent fab16624
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment