Merge pull request #1407 from microsoftgraph/dependabot/gradle/v3/longTermBranch/com.github.spotbugs-6.0.4

chore(deps): bump com.github.spotbugs from 6.0.2 to 6.0.4

Author: baywet

build.gradle

@@ -5,7 +5,7 @@ plugins {
     id 'maven-publish'
     id 'signing'
     id 'jacoco'
-    id 'com.github.spotbugs' version '6.0.2'
+    id 'com.github.spotbugs' version '6.0.4'
     id "org.sonarqube" version "4.4.1.3373"
 }
 

src/main/java/com/microsoft/graph/core/requests/BaseGraphRequestAdapter.java

@@ -153,11 +153,14 @@ public BaseGraphRequestAdapter(@Nonnull AuthenticationProvider authenticationPro
         this(authenticationProvider, determineBaseAddress(cloud, version), graphClientOption);
     }
 
-    private static String determineBaseAddress(@Nullable Clouds nationalCloud, @Nullable String version) throws IllegalArgumentException {
-        String cloud = nationalCloud == null ? getCloudList().get(Clouds.GLOBAL_CLOUD) : getCloudList().get(nationalCloud);
+    private static String determineBaseAddress(@Nullable final Clouds nationalCloud, @Nullable final String version) {
+        final String cloud = nationalCloud == null ? getCloudList().get(Clouds.GLOBAL_CLOUD) : getCloudList().get(nationalCloud);
         if(cloud == null) {
             throw new IllegalArgumentException(nationalCloud+" is an unexpected national cloud.");
         }
         return version == null ? (cloud+"/v1.0") : (cloud+"/"+version);
     }
+    protected final void finalize() throws Throwable{
+        // to avoid finalizer attacks on the constructors defensive programming
+    }
 }