You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2022/05/27 15:02:29 UTC
[GitHub] [commons-lang] dependabot[bot] opened a new pull request, #902: Bump spotbugs from 4.6.0 to 4.7.0
dependabot[bot] opened a new pull request, #902:
URL: https://github.com/apache/commons-lang/pull/902
Bumps [spotbugs](https://github.com/spotbugs/spotbugs) from 4.6.0 to 4.7.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/spotbugs/spotbugs/releases">spotbugs's releases</a>.</em></p>
<blockquote>
<h2>SpotBugs 4.7.0</h2>
<h2>CHANGELOG</h2>
<h3>Changed</h3>
<ul>
<li>Updated documentation by adding parenthesis <code>()</code> to the negative odd check message (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1995">#1995</a>) <a href="https://github.com/axkr"><code>@axkr</code></a></li>
<li>Let the Plugin class implement AutoCloseable so we can release the .jar file (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/2024">#2024</a>) <a href="https://github.com/gtoison"><code>@gtoison</code></a></li>
</ul>
<h3>Fixed</h3>
<ul>
<li>Fixed reports to truncate existing files before writing new content (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1950">#1950</a>) <a href="https://github.com/sdati"><code>@sdati</code></a></li>
<li>Fixed traversal of nested archives governed by <code>-nested:true</code> (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1930">#1930</a>) <a href="https://github.com/Vogel612"><code>@Vogel612</code></a></li>
<li>Warnings of deprecated System::setSecurityManager calls on Java 17 (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1983">#1983</a>) <a href="https://github.com/wborn"><code>@wborn</code></a></li>
<li>Fixed false positive SSD bug for locking on java.lang.Class objects (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1978">#1978</a>) <a href="https://github.com/jpschewe"><code>@jpschewe</code></a></li>
<li>FindReturnRef throws an IllegalArgumentException unexpectedly (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/2019">#2019</a>) <a href="https://github.com/KengoTODA"><code>@KengoTODA</code></a></li>
<li>Bumped Saxon-HE from 10.6 to 11.3 (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1955">#1955</a>, <a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1999">#1999</a>)</li>
<li>Bump ObjectWeb ASM from 9.2 to 9.3 supporting JDK 19 (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/2004">#2004</a>)</li>
</ul>
<h3>Added</h3>
<ul>
<li>New detector <code>ThrowingExceptions</code> and introduced new bug types <a href="https://github.com/oroszbd"><code>@oroszbd</code></a>
<ul>
<li><code>THROWS_METHOD_THROWS_RUNTIMEEXCEPTION</code> is reported in case of a method throwing RuntimeException,</li>
<li><code>THROWS_METHOD_THROWS_CLAUSE_BASIC_EXCEPTION</code> is reported when a method has Exception in its throws clause and</li>
<li><code>THROWS_METHOD_THROWS_CLAUSE_THROWABLE</code> is reported when a method has Throwable in its throws clause (See <a href="https://wiki.sei.cmu.edu/confluence/display/java/ERR07-J.+Do+not+throw+RuntimeException%2C+Exception%2C+or+Throwable">SEI CERT ERR07-J</a>)</li>
</ul>
</li>
<li>New rule <code>PERM_SUPER_NOT_CALLED_IN_GETPERMISSIONS</code> to warn for custom class loaders who do not call their superclasses' <code>getPermissions()</code> in their <code>getPermissions()</code> method. This rule based on the SEI CERT rule <em>SEC07-J Call the superclass's getPermissions() method when writing a custom class loader</em>. (<a href="https://wiki.sei.cmu.edu/confluence/display/java/SEC07-J.+Call+the+superclass%27s+getPermissions%28%29+method+when+writing+a+custom+class+loader">#SEC07-J</a>) <a href="https://github.com/baloghadamsoftware"><code>@baloghadamsoftware</code></a></li>
<li>New rule <code>USC_POTENTIAL_SECURITY_CHECK_BASED_ON_UNTRUSTED_SOURCE</code> to detect cases where a non-final method of a non-final class is called from public methods of public classes and then the same method is called on the same object inside a doPrivileged block. Since the called method may have been overridden to behave differently on the first and second invocations this is a possible security check based on an unreliable source. This rule is based on <em>SEC02-J. Do not base security checks on untrusted sources</em>. (<a href="https://wiki.sei.cmu.edu/confluence/display/java/SEC02-J.+Do+not+base+security+checks+on+untrusted+sources">#SEC02-J</a>) <a href="https://github.com/baloghadamsoftware"><code>@baloghadamsoftware</code></a></li>
<li>New detector <code>DontUseFloatsAsLoopCounters</code> to detect usage of floating-point variables as loop counters (<code>FL_FLOATS_AS_LOOP_COUNTERS</code>), according to SEI CERT rules <a href="https://wiki.sei.cmu.edu/confluence/display/java/NUM09-J.+Do+not+use+floating-point+variables+as+loop+counters">NUM09-J. Do not use floating-point variables as loop counters</a> <a href="https://github.com/adrianturtoczki"><code>@adrianturtoczki</code></a></li>
<li>New test detector <code>ViewCFG</code> to visualize the control-flow graph for <code>SpotBugs</code> developers <a href="https://github.com/baloghadamsoftware"><code>@baloghadamsoftware</code></a></li>
</ul>
<h2>CHECKSUM</h2>
<table>
<thead>
<tr>
<th>file</th>
<th>checksum (sha256)</th>
</tr>
</thead>
<tbody>
<tr>
<td>spotbugs-4.7.0-javadoc.jar</td>
<td>43745221e8fdf50fa2f89d659034523dd58da3d10223de6e0c91704c07e025a0</td>
</tr>
<tr>
<td>spotbugs-4.7.0-sources.jar</td>
<td>6e90f856826b48a3031e2cb903534b4fdb494759863ea14e8df93c9cf15a272c</td>
</tr>
<tr>
<td>spotbugs-4.7.0.tgz</td>
<td>8c871e279c7d9b1933158db6355b8ac817a84fd724b88b1e393e3abcf6874910</td>
</tr>
<tr>
<td>spotbugs-4.7.0.zip</td>
<td>9ee793b0f3f78901089211dfa67b8603e38fd1abd64eac09d2590df506cfedf5</td>
</tr>
<tr>
<td>spotbugs-annotations-4.7.0-javadoc.jar</td>
<td>76a9a7d45590494a220840d173809b2fe0ec50e554435dd9b28de9312cc6a34a</td>
</tr>
<tr>
<td>spotbugs-annotations-4.7.0-sources.jar</td>
<td>b338136e3e82d585348cde58a8fe3a678e16f51a35c31c1463e05fefef557aad</td>
</tr>
<tr>
<td>spotbugs-annotations.jar</td>
<td>8e6677102aa0de50841644cf9a57d6d503550ad774049a76c75bf157a8beebd5</td>
</tr>
<tr>
<td>spotbugs-ant-4.7.0-javadoc.jar</td>
<td>50d84857bbdca54898e93b225835ab25eba9f0a7e340c420ab08bc17bd584f0c</td>
</tr>
<tr>
<td>spotbugs-ant-4.7.0-sources.jar</td>
<td>ce7cfbed848ccb0e3765cec6b9c60c458699aa51f60ad9216cf89dbf38d8d793</td>
</tr>
<tr>
<td>spotbugs-ant.jar</td>
<td>b866a2a89a03b49e60b5f27e0f5987eb8c12c2d2aefc6e9ddcbcdae345c765db</td>
</tr>
<tr>
<td>spotbugs.jar</td>
<td>94c44d86c83b8fe63b20023e510874aef721b1081982051706e1da841572f295</td>
</tr>
<tr>
<td>test-harness-4.7.0-javadoc.jar</td>
<td>3af01af49d74964569fe8ce0e4c217fbfd89e7c02ae5428148b5222e0aec8906</td>
</tr>
<tr>
<td>test-harness-4.7.0-sources.jar</td>
<td>7efb06093ea5f6f330a7bd76b894f396d6cb466665fcefc01a3743b07910dc29</td>
</tr>
<tr>
<td>test-harness-4.7.0.jar</td>
<td>50b4a72c668ea7d29bf1234b4aa380df903374216f68b0a87f7ca28d4fa225f3</td>
</tr>
<tr>
<td>test-harness-core-4.7.0-javadoc.jar</td>
<td>659508cc31a9dfb5c7d4c14981a3f38f476888c7bb08479ac19401ef39201f64</td>
</tr>
<tr>
<td>test-harness-core-4.7.0-sources.jar</td>
<td>f8aab3c5cdd456d6b6d632e9fc65897e657447a2e925b6b3f61bd2d15c22cb24</td>
</tr>
<tr>
<td>test-harness-core-4.7.0.jar</td>
<td>7165f7f45a6e82e8a6d6a0a4033b6473b310c14f645cb62ebc2fbc6ce5338350</td>
</tr>
<tr>
<td>test-harness-jupiter-4.7.0-javadoc.jar</td>
<td>c0300bac23ce2292be120325515128d2d9262c1f7d71b3bf1c4324b2d6b57753</td>
</tr>
<tr>
<td>test-harness-jupiter-4.7.0-sources.jar</td>
<td>210353a57016e26b1a654d936a15f039613fa1ac532d485c1b1d03902f6c6315</td>
</tr>
<tr>
<td>test-harness-jupiter-4.7.0.jar</td>
<td>18095fec31b85981ecaafdef86ca9ae1e9588e1b9bc6d209f82829cf9d0c13f4</td>
</tr>
</tbody>
</table>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a href="https://github.com/spotbugs/spotbugs/blob/master/CHANGELOG.md">spotbugs's changelog</a>.</em></p>
<blockquote>
<h2>4.7.0 - 2022-04-14</h2>
<h3>Changed</h3>
<ul>
<li>Updated documentation by adding parenthesis <code>()</code> to the negative odd check message (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1995">#1995</a>)</li>
<li>Let the Plugin class implement AutoCloseable so we can release the .jar file (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/2024">#2024</a>)</li>
</ul>
<h3>Fixed</h3>
<ul>
<li>Fixed reports to truncate existing files before writing new content (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1950">#1950</a>)</li>
<li>Bumped Saxon-HE from 10.6 to 11.3 (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1955">#1955</a>, <a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1999">#1999</a>)</li>
<li>Fixed traversal of nested archives governed by <code>-nested:true</code> (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1930">#1930</a>)</li>
<li>Warnings of deprecated System::setSecurityManager calls on Java 17 (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/1983">#1983</a>)</li>
<li>Fixed false positive SSD bug for locking on java.lang.Class objects (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1978">#1978</a>)</li>
<li>FindReturnRef throws an IllegalArgumentException unexpectedly (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/2019">#2019</a>)</li>
<li>Bump ObjectWeb ASM from 9.2 to 9.3 supporting JDK 19 (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/pull/2004">#2004</a>)</li>
</ul>
<h3>Added</h3>
<ul>
<li>New detector <code>ThrowingExceptions</code> and introduced new bug types:
<ul>
<li><code>THROWS_METHOD_THROWS_RUNTIMEEXCEPTION</code> is reported in case of a method throwing RuntimeException,</li>
<li><code>THROWS_METHOD_THROWS_CLAUSE_BASIC_EXCEPTION</code> is reported when a method has Exception in its throws clause and</li>
<li><code>THROWS_METHOD_THROWS_CLAUSE_THROWABLE</code> is reported when a method has Throwable in its throws clause (See <a href="https://wiki.sei.cmu.edu/confluence/display/java/ERR07-J.+Do+not+throw+RuntimeException%2C+Exception%2C+or+Throwable">SEI CERT ERR07-J</a>)</li>
</ul>
</li>
<li>New rule <code>PERM_SUPER_NOT_CALLED_IN_GETPERMISSIONS</code> to warn for custom class loaders who do not call their superclasses' <code>getPermissions()</code> in their <code>getPermissions()</code> method. This rule based on the SEI CERT rule <em>SEC07-J Call the superclass's getPermissions() method when writing a custom class loader</em>. (<a href="https://wiki.sei.cmu.edu/confluence/display/java/SEC07-J.+Call+the+superclass%27s+getPermissions%28%29+method+when+writing+a+custom+class+loader">#SEC07-J</a>)</li>
<li>New rule <code>USC_POTENTIAL_SECURITY_CHECK_BASED_ON_UNTRUSTED_SOURCE</code> to detect cases where a non-final method of a non-final class is called from public methods of public classes and then the same method is called on the same object inside a doPrivileged block. Since the called method may have been overridden to behave differently on the first and second invocations this is a possible security check based on an unreliable source. This rule is based on <em>SEC02-J. Do not base security checks on untrusted sources</em>. (<a href="https://wiki.sei.cmu.edu/confluence/display/java/SEC02-J.+Do+not+base+security+checks+on+untrusted+sources">#SEC02-J</a>)</li>
<li>New detector <code>DontUseFloatsAsLoopCounters</code> to detect usage of floating-point variables as loop counters (<code>FL_FLOATS_AS_LOOP_COUNTERS</code>), according to SEI CERT rules <a href="https://wiki.sei.cmu.edu/confluence/display/java/NUM09-J.+Do+not+use+floating-point+variables+as+loop+counters">NUM09-J. Do not use floating-point variables as loop counters</a></li>
<li>New test detector <code>ViewCFG</code> to visualize the control-flow graph for <code>SpotBugs</code> developers</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/spotbugs/spotbugs/commit/c6bff34f4eaf53c9f4cfb2d4973b5cc8c5239248"><code>c6bff34</code></a> release v4.7.0</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/38d20299bac08aea013f83f8e6d7b9021c98e61b"><code>38d2029</code></a> docs: bump up version of findsecbugs-plugin in the doc</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/4e61eacfaedc2319832e3ae2ef74da5c985afec1"><code>4e61eac</code></a> fix: let Plugin implement AutoCloseable</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/9725e34f33cf76968422c36b0611ec08b9ced98f"><code>9725e34</code></a> CFG Visualizer (implemented as a test detector) (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/2014">#2014</a>)</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/85ebe285cfab4861054cdbd7f9c3d961774e68e5"><code>85ebe28</code></a> Added new detector for NUM09-J: Do not use floating-point variables as loop c...</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/23f37b977da8dce96002e925aa40e490f7b51c10"><code>23f37b9</code></a> Fix report output to truncate existing files (<a href="https://github-redirect.dependabot.com/spotbugs/spotbugs/issues/1951">#1951</a>)</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/beec2f529324cf0a276540c5528e7c23c94a4d75"><code>beec2f5</code></a> build(deps): bump goomph from 3.36.1 to 3.36.2 in /buildSrc</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/79aba05c83b7bb25409a0532806a33b6f6c268a6"><code>79aba05</code></a> build(deps): bump checker-qual from 3.21.4 to 3.22.0</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/fda53ce4790cb7a1923d2102cbf35af7086655a4"><code>fda53ce</code></a> build(deps): bump mockito-core from 4.5.0 to 4.5.1</li>
<li><a href="https://github.com/spotbugs/spotbugs/commit/1c745634b21756335a60c52112501d7759f7ba23"><code>1c74563</code></a> build(deps): bump goomph from 3.36.0 to 3.36.1 in /buildSrc</li>
<li>Additional commits viewable in <a href="https://github.com/spotbugs/spotbugs/compare/4.6.0...4.7.0">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] dependabot[bot] commented on pull request #902: Bump spotbugs from 4.6.0 to 4.7.0
Posted by GitBox <gi...@apache.org>.
dependabot[bot] commented on PR #902:
URL: https://github.com/apache/commons-lang/pull/902#issuecomment-1172441540
Superseded by #917.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] codecov-commenter commented on pull request #902: Bump spotbugs from 4.6.0 to 4.7.0
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #902:
URL: https://github.com/apache/commons-lang/pull/902#issuecomment-1139704320
# [Codecov](https://codecov.io/gh/apache/commons-lang/pull/902?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#902](https://codecov.io/gh/apache/commons-lang/pull/902?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (7d0d971) into [master](https://codecov.io/gh/apache/commons-lang/commit/6eb223bb8c68540e3f2bfa08c3d87b4cf7dfb544?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (6eb223b) will **not change** coverage.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## master #902 +/- ##
=========================================
Coverage 91.41% 91.41%
Complexity 7390 7390
=========================================
Files 188 188
Lines 15967 15967
Branches 2998 2998
=========================================
Hits 14596 14596
Misses 764 764
Partials 607 607
```
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/commons-lang/pull/902?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/commons-lang/pull/902?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [6eb223b...7d0d971](https://codecov.io/gh/apache/commons-lang/pull/902?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] dependabot[bot] closed pull request #902: Bump spotbugs from 4.6.0 to 4.7.0
Posted by GitBox <gi...@apache.org>.
dependabot[bot] closed pull request #902: Bump spotbugs from 4.6.0 to 4.7.0
URL: https://github.com/apache/commons-lang/pull/902
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org