Fixed Spotbugs Switch related issues and added Suppresses

[JENKINS-60625] Prevent Jenkins page rendering from being blocked when the update center data parsing is in progress

* removed redundnat null checks of nonnull values

* integrated suggestions

* removed local jenkins variable as suggested

* replaced null check with check for error object

* IntelliJ/Java: Duplicate declaration

AntClassLoader.getUrl

* Add missing import for annotation

* Switch from Vector to ArrayList<File>

* Drop inferable File

* Use for (File pathComponent : pathComponents) sugar

* Accept generic container type for getUrl
Co-Authored-By: NOleg Nenashev <o.v.nenashev@gmail.com>
Co-authored-by: NOleg Nenashev <o.v.nenashev@gmail.com>

* Add new module optional-depender and a new failing test

* Variant plugin is a dependency of optional-depender

Optional-depender contains the following class

@OptionalExtension(requirePlugins = {"dependee"})
public class OptionalDependerExtension {
      private static void foo(Dependee d) {

      }
}

* [JENKINS-60449] Move plugin initialization at the end, keep only one extension refresh

* Improve logic to detect classloading issues

Inspect all declared constructors/methods/fields recursively to exclude
failing classes from Guice rather than have Guice explode.

This also fix the failing test, but the previous commit is still better
because it avoids an extra Jenkins.get().refreshExtensions() call.

* Only test resolveClass method if the classloader is not the bootstrap one

Co-Authored-By: NDaniel Beck <1831569+daniel-beck@users.noreply.github.com>

Spotbugs: remove UMAC_UNCALLABLE_METHOD_OF_ANONYMOUS_CLASS from global exclude

set spotbugs URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD to ignore because reads are not detectable by spotbugs

* Add Jenkins Changelog Generator to the GitHub Actions release drafter Pipeline

* The Changelog generator repo was renamed

* Switch Release Drafter to GitHub Actions

* Switch back to the upstream version after fixes

* [JENKINS-21311] Make the queue/cancelItem API an API.

This change:
* documents the API
* adds meaningful return codes to help API caller.

* [JENKINS-21311] Code review

* [JENKINS-21311] Update failing integration test

* [JENKINS-36720]: Fixed Spotbugs issues WMI_WRONG_MAP_ITERATOR and OS_OPEN_STREAM in core

* merged try with resources with try finally

* reverted auto format from IDE

* reverted auto format from IDE

* use Util.fileToPath() as suggested to remove catch block

* merged try statements

* merged try statements

* reverted some whitespace fixes

* reverted another whitespace fix

* Apply suggestions from code review
Co-Authored-By: NOleg Nenashev <o.v.nenashev@gmail.com>

* reverted change

* reverted change

* reverted change
Co-authored-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Remove unused direct dependency commons-codec from Jenkins CLI

* replaced implementation with call to Integer.compare or Long.compare

* Update core/src/main/java/hudson/model/HealthReport.java
Co-Authored-By: NZbynek Konecny <zbynek1729@gmail.com>

* mark compare methods as deprecated
Co-authored-by: NZbynek Konecny <zbynek1729@gmail.com>

removed obsolete SpotBugs suppresses

* JENKINS-60513: Added AtomicInteger to whitelist

* JENKINS-60513: Mark field as transient

* reverted whitelist change

* renamed field to support deserialization of ols values

* refactor exception to actually put the failing string inside instead of null

* do not catch RuntimeExceptions

* Adapt to newer HtmlUnit.

* https://github.com/HtmlUnit/htmlunit/pull/29 seems to have been incompatible.
WebClient.addRequestHeader will no longer override a header in a WebRequest,
such as when the same WebClient/WebRequest was previously used with different headers.

* Tracked down a behavioral change in passing through URL-encoded path characters.
https://github.com/HtmlUnit/htmlunit/commit/2c4956863420e4baef9d3d8c23ec0577ec64d2bf picks up
https://github.com/apache/httpcomponents-client/commit/8c04c6ae5e5ba1432e40684428338ce68431766b which is the actual change.

* jenkins-test-harness 2.58

[JENKINS-59107] User is no longer logged out when authenticating another user

fixed_typos_in_bom/pom.xml

[JENKINS-60351] - Make it possible to disable multiple deletion attempts by setting hudson.Util.maxFileDeletionRetries to zero (#4382)

* [JENKINS-60351] It should be possible to disable a retry of the
PathRemover

This does not address the issues that the comments and code in
hudons.Util are inconsistent if they are for retries or total number of
attempts (they are currently total number of attempts)

* [JENKINS-60351] Make the code conform to the documentation

maxRetries should control the number of retries not the number of
attempts.

* rename field from DELETION_MAX to DELETION_RETRIES

DNS comparisons of jenkins.io and www.jenkins.io do not pass.

Fixed typos in comment on servlet-api version tag

This fixes a regression introduced in SECURITY-901 where performing an authentication check on another user while logged in caused the user to get logged out.
Signed-off-by: NMatt Sicker <boards@gmail.com>

* replaced integer with AtomicInteger to fix spotbugs issue

* added @Restricted to prevent further usage

* replaced volatile with final because of AtomicInteger usage

* implemented a getter and an increment method

* fixed typeo in javadoc