Branch: refs/heads/master Home: https://github.com/jenkinsci/remoting Commit: 6000a6f0eef6ed71c66ee07ed076395cc8f65caf https://github.com/jenkinsci/remoting/commit/6000a6f0eef6ed71c66ee07ed076395cc8f65caf Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-10 (Tue, 10 Nov 2015)
Changed paths: M src/main/java/hudson/remoting/ClassFilter.java A src/test/java/hudson/remoting/DefaultClassFilterTest.java Log Message: ----------- [SECURITY-218] Allow a user to specificy the blacklisted classes. We want a user to be able to blacklist classes without unpacking wars or changing code, so provide a mechanism for an administrator to set a property (hudson.remoting.ClassFilter.DEFAULTS_OVERRIDE_LOCATION) to point at a file that contains a list of regular expressions. A match in the RegExp signifies that the class should be blacklisted. Commit: e00669b08ba60a1822833a0e96bc384726fb4d13 https://github.com/jenkinsci/remoting/commit/e00669b08ba60a1822833a0e96bc384726fb4d13 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-10 (Tue, 10 Nov 2015) Changed paths: M pom.xml M src/main/java/hudson/remoting/ChannelBuilder.java M src/main/java/hudson/remoting/ClassFilter.java M src/main/java/hudson/remoting/ObjectInputStreamEx.java M src/test/java/hudson/remoting/DefaultClassFilterTest.java Log Message: ----------- fix the codes use of the public static. make sure we are using Linux line ends. workround bug in javac (if you import org.hamcrest.BaseMatcher then the compilation fails as if it can not resolve BaseMatcher) Commit: 1d59ff3ca1da35a062acf11a49a678a2d0456f21 https://github.com/jenkinsci/remoting/commit/1d59ff3ca1da35a062acf11a49a678a2d0456f21 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-10 (Tue, 10 Nov 2015) Changed paths: M pom.xml Log Message: ----------- hamcrest shoukd be test scope Commit: 9e2adc60ec2dd48a8da80b69caef748c2775c800 https://github.com/jenkinsci/remoting/commit/9e2adc60ec2dd48a8da80b69caef748c2775c800 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M pom.xml Log Message: ----------- fix broken pom Commit: 7621980ea4ae6b50ac0883fb1513cdb004909d92 https://github.com/jenkinsci/remoting/commit/7621980ea4ae6b50ac0883fb1513cdb004909d92 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- remove volatile from ClassFilter.DEFAULT Commit: 88e658101875cc9755e590911a426e8bf49fe893 https://github.com/jenkinsci/remoting/commit/88e658101875cc9755e590911a426e8bf49fe893 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- Fix logging lines containing 'load load' Commit: a865b02d15c2cf7ee4bd05b9ef24f1f7ee5c8842 https://github.com/jenkinsci/remoting/commit/a865b02d15c2cf7ee4bd05b9ef24f1f7ee5c8842 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- add CheckForNull to helper function. Commit: b7a5f494d93d23c1ff2c690d1a67687bc807ec66 https://github.com/jenkinsci/remoting/commit/b7a5f494d93d23c1ff2c690d1a67687bc807ec66 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M src/test/java/hudson/remoting/DefaultClassFilterTest.java Log Message: ----------- Add copyright statement. Commit: 4717016bf449a44c075d8e55c81615906c619543 https://github.com/jenkinsci/remoting/commit/4717016bf449a44c075d8e55c81615906c619543 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: A src/test/java/hudson/remoting/RegExpBenchmark.java Log Message: ----------- added a benchmark as requested by @oleg-nenashev Commit: dd034a110175e29a0e30691ad996742649eb1fe5 https://github.com/jenkinsci/remoting/commit/dd034a110175e29a0e30691ad996742649eb1fe5 Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M src/test/java/hudson/remoting/RegExpBenchmark.java Log Message: ----------- fix benchmark and optimize regexp Commit: 647f76bd6193c668d754358f2c77cea76f25e49d https://github.com/jenkinsci/remoting/commit/647f76bd6193c668d754358f2c77cea76f25e49d Author: James Nord <jnord+git...@cloudbees.com> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- hudson.cli.Connection uses ClassFilter.Default. @jglick notes that it is safer to retain the backwards compatability than to fix the above class to use the new code. Commit: 9ce77a5bd3e412925ceb6901fcabde066e3eb1aa https://github.com/jenkinsci/remoting/commit/9ce77a5bd3e412925ceb6901fcabde066e3eb1aa Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M pom.xml M src/main/java/hudson/remoting/ChannelBuilder.java M src/main/java/hudson/remoting/ClassFilter.java M src/main/java/hudson/remoting/ObjectInputStreamEx.java A src/test/java/hudson/remoting/DefaultClassFilterTest.java A src/test/java/hudson/remoting/RegExpBenchmark.java Log Message: ----------- Merge branch 'SECURITY-218-customizable-blacklist' into 2.53.x This came from https://github.com/jenkinsci-cert/remoting/pull/4 but it was accepted into 2.53.2 as a part of 1.625.3 Commit: 937fa06f72cd55d427e92571423d30d1f2838ff8 https://github.com/jenkinsci/remoting/commit/937fa06f72cd55d427e92571423d30d1f2838ff8 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ChannelBuilder.java M src/main/java/hudson/remoting/ClassFilter.java M src/main/java/hudson/remoting/ObjectInputStreamEx.java M src/test/java/hudson/remoting/DefaultClassFilterTest.java Log Message: ----------- Fixed the signature change in ClassFilter.NOOP I get that James doesn't like public static final fields, but in this case I think the change is overzealous. Let's respect how this was originally done and stick to it. There are multiple good reasons. There's a principle of minimizing a change for security fixes. This fix is allowed to get into 1.625.3 through a slip-stream, so let's not make changes that are not necessary. Then there's added complexity in keeping the old signature working while adding the new getDefaultFilter() method. The code as it was also removed the final modifier from the field, which is not nice. There's also no good reason why constants like NONE is ever going to be something other than constants. All in all I think the fix is not worth the complication at this point. Commit: 813962ca09cbf970e3e9bbb8606b6d46d61a1851 https://github.com/jenkinsci/remoting/commit/813962ca09cbf970e3e9bbb8606b6d46d61a1851 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- INFO logging interferes with console apps that uses remoting. JVM puts INFO logging to stdout by default, so if it's used by a CLI app that needs to precisely control its output, it'll mess that up. This includes Jenkins CLI. To make up for the loss of ability to inspect what the actual rules are, add the toString() method, so that for example Jenkins users can run "print ClassFilter.DEFAULT" to find out the actual rules, not just that the custom properties are in use. Commit: d7e780ca99977762f0966b7b5a734627d54856ad https://github.com/jenkinsci/remoting/commit/d7e780ca99977762f0966b7b5a734627d54856ad Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- Better to fail instead of fall back. If the user did specify an override file, it's better to fail hard than fall back to the default in case of a problem. The situation I'm afraid of is that when Jenkins is using this, you can make an erranous change to the override file and fail to realize that your configuration change is not taking effect. Noticing that requires an active act of looking into log files, which people don't do. Commit: 6147555e8a02142196cf0ac27b22bce39f6a80ca https://github.com/jenkinsci/remoting/commit/6147555e8a02142196cf0ac27b22bce39f6a80ca Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- Make it obvious that 'return null' is an error case Accidental fall through doesn't happen in this way, and nesting level gets less. Commit: aa4901c5f85e62a3617222bda42cf00deec4de2f https://github.com/jenkinsci/remoting/commit/aa4901c5f85e62a3617222bda42cf00deec4de2f Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java M src/test/java/hudson/remoting/DefaultClassFilterTest.java Log Message: ----------- Fixed up test cases Commit: e5f18b4f10cdbb99a3206f80f3b5c76b55186243 https://github.com/jenkinsci/remoting/commit/e5f18b4f10cdbb99a3206f80f3b5c76b55186243 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M src/main/java/hudson/remoting/ClassFilter.java Log Message: ----------- Fixing findbugs warning. This is a difficult call. but in the environment where the platform encoding is not UTF-8, users expect to write such files in the platform default encoding. The downside is that the definition file is now non-portable, though non-ASCII class names are uncommon even among CJK users, and this file is really only going to be used in Jenkins masters, so portability is less important compared to something that needs to be on all slaves. Commit: 2b4c8708a871faa84b483f4b757cd3c8a817d6cb https://github.com/jenkinsci/remoting/commit/2b4c8708a871faa84b483f4b757cd3c8a817d6cb Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M pom.xml Log Message: ----------- [maven-release-plugin] prepare release remoting-2.53.2 Commit: db9c9d913b59e2b9a6f26d9936fb36bddd8492e1 https://github.com/jenkinsci/remoting/commit/db9c9d913b59e2b9a6f26d9936fb36bddd8492e1 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M pom.xml Log Message: ----------- [maven-release-plugin] prepare for next development iteration Commit: 1e68eb88f1f9c9a7ad76270d54f703148c947476 https://github.com/jenkinsci/remoting/commit/1e68eb88f1f9c9a7ad76270d54f703148c947476 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2015-11-25 (Wed, 25 Nov 2015) Changed paths: M pom.xml M src/main/java/hudson/remoting/ClassFilter.java A src/test/java/hudson/remoting/DefaultClassFilterTest.java A src/test/java/hudson/remoting/RegExpBenchmark.java Log Message: ----------- Merge branch '2.53.x' Conflicts: pom.xml Compare: https://github.com/jenkinsci/remoting/compare/47ae12122cdd...1e68eb88f1f9 -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.