Skip to content

Commit

Permalink
Merge pull request jenkinsci#328 from jenkinsci/jglick-patch-2
Browse files Browse the repository at this point in the history
Noting XStream issue
  • Loading branch information
oleg-nenashev authored Nov 7, 2020
2 parents 978b3b3 + 2da9ae1 commit f52f17b
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions jep/228/README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,8 @@ If link:../7/README.adoc[JEP-7] is implemented, then `XStream2.setMapper` and `.

Maintaining a custom fork of a library was an additional maintenance burden for Jenkins developers.
The fork was well out of date, leading to suspicions that security hardenings added in recent years may be missing, though no vulnerabilities are known.
Jenkins was also missing out on potentially useful improvements, such as
link:https://github.com/x-stream/xstream/issues/101[better Java 11 compatibility].

A key reason for maintaining the fork was to introduce thread safety as well as concurrency-optimized collections into XStream.
In fact upstream XStream is designed to be thread-safe _after_ its configuration has been frozen.
Expand Down

0 comments on commit f52f17b

Please sign in to comment.