Merge pull request #518 from microsoftgraph/dev

baywet · web-flow · commit 2e374d8aa0a1 · 2022-05-26T11:27:11.000-07:00
2.0.13
diff --git a/.github/workflows/build-and-publish.yml b/.github/workflows/build-and-publish.yml
@@ -105,7 +105,7 @@ jobs:
         run: .\scripts\getLatestVersion.ps1
         shell: pwsh
       - name: Create tag
-        uses: rickstaa/action-create-tag@v1.2.2
+        uses: rickstaa/action-create-tag@v1.3.2
         with:
           tag: ${{ steps.GetVersion.outputs.tag }}
       - name: Queue Git Release
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
@@ -35,15 +35,15 @@ jobs:
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v1
+      uses: github/codeql-action/init@v2
       # Override language selection by uncommenting this and choosing your languages
       with:
         languages: java
 
     # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
     # If this step fails, then you should remove it and run the build manually (see below)
     - name: Autobuild
-      uses: github/codeql-action/autobuild@v1
+      uses: github/codeql-action/autobuild@v2
 
     # ℹ️ Command-line programs to run using the OS shell.
     # 📚 https://git.io/JvXDl
@@ -57,4 +57,4 @@ jobs:
     #   make release
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v1
+      uses: github/codeql-action/analyze@v2
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
+## [2.0.13] - 2022-05-26
+
+### Added
+
+### Changed
+
+- Fixed an issue where the error message would be logged twice. #514
+- Bumps azure-core to 1.28.0 #503, #504, #506
+- Bumps azure-identity to 1.5.1 #505, #507
+
 ## [2.0.12] - 2022-04-22
 
 ### Added
diff --git a/android/build.gradle b/android/build.gradle
@@ -8,8 +8,8 @@ buildscript {
     }
 
     dependencies {
-        classpath "com.gradle:gradle-enterprise-gradle-plugin:3.10"
-        classpath "com.android.tools.build:gradle:7.1.3"
+        classpath "com.gradle:gradle-enterprise-gradle-plugin:3.10.1"
+        classpath "com.android.tools.build:gradle:7.2.1"
         classpath "com.github.ben-manes:gradle-versions-plugin:0.42.0"
     }
 }
diff --git a/build.gradle b/build.gradle
@@ -14,7 +14,7 @@ plugins {
     id 'maven-publish'
     id 'signing'
     id 'jacoco'
-    id 'com.github.spotbugs' version '5.0.6'
+    id 'com.github.spotbugs' version '5.0.7'
     id "org.sonarqube" version "3.3"
 }
 
diff --git a/gradle/dependencies.gradle b/gradle/dependencies.gradle
@@ -10,10 +10,10 @@ dependencies {
     implementation 'com.google.guava:guava:31.1-jre'
 
     implementation 'com.google.code.gson:gson:2.9.0'
-    api 'com.azure:azure-core:1.27.0'
+    api 'com.azure:azure-core:1.28.0'
 
     compileOnly 'net.jcip:jcip-annotations:1.0'
-    compileOnly 'com.github.spotbugs:spotbugs-annotations:4.6.0'
+    compileOnly 'com.github.spotbugs:spotbugs-annotations:4.7.0'
     testCompileOnly 'net.jcip:jcip-annotations:1.0'
-    testCompileOnly 'com.github.spotbugs:spotbugs-annotations:4.5.3'
+    testCompileOnly 'com.github.spotbugs:spotbugs-annotations:4.7.0'
 }
diff --git a/pom.xml b/pom.xml
@@ -35,7 +35,7 @@
         <dependency>
             <groupId>com.azure</groupId>
             <artifactId>azure-core</artifactId>
-            <version>1.27.0</version>
+            <version>1.28.0</version>
         </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
diff --git a/samples/deviceCodeSample/build.gradle b/samples/deviceCodeSample/build.gradle
@@ -12,5 +12,5 @@ repositories {
 dependencies {
     testImplementation group: 'junit', name: 'junit', version: '4.13.2'
     implementation project(':coreLibrary')
-    implementation 'com.azure:azure-identity:1.5.0'
+    implementation 'com.azure:azure-identity:1.5.1'
 }
diff --git a/samples/interactiveBrowserSample/build.gradle b/samples/interactiveBrowserSample/build.gradle
@@ -12,5 +12,5 @@ repositories {
 dependencies {
     testImplementation group: 'junit', name: 'junit', version: '4.13.2'
     implementation project(':coreLibrary')
-    implementation 'com.azure:azure-identity:1.5.0'
+    implementation 'com.azure:azure-identity:1.5.1'
 }
diff --git a/src/main/java/com/microsoft/graph/http/CoreHttpProvider.java b/src/main/java/com/microsoft/graph/http/CoreHttpProvider.java
@@ -28,7 +28,6 @@
 import com.microsoft.graph.httpcore.middlewareoption.RedirectOptions;
 import com.microsoft.graph.httpcore.middlewareoption.RetryOptions;
 import com.microsoft.graph.logger.ILogger;
-import com.microsoft.graph.logger.LoggerLevel;
 import com.microsoft.graph.options.HeaderOption;
 import com.microsoft.graph.serializer.ISerializer;
 
@@ -489,8 +488,7 @@ private <Result, Body, DeserializeType> Result processResponse(final Response re
                 }
             }
         } catch (final GraphServiceException ex) {
-            final boolean shouldLogVerbosely = logger.getLoggingLevel() == LoggerLevel.DEBUG;
-            logger.logError("Graph service exception " + ex.getMessage(shouldLogVerbosely), ex);
+            logger.logError("Graph service exception", ex);
             throw ex;
         } catch (final Exception ex) {
             final ClientException clientException = new ClientException("Error during http request",
diff --git a/src/main/java/com/microsoft/graph/http/IStatefulResponseHandler.java b/src/main/java/com/microsoft/graph/http/IStatefulResponseHandler.java
@@ -22,9 +22,9 @@
 
 package com.microsoft.graph.http;
 
-
 import com.microsoft.graph.logger.ILogger;
 import com.microsoft.graph.serializer.ISerializer;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 import javax.annotation.Nullable;
 import javax.annotation.Nonnull;
@@ -50,6 +50,7 @@ public interface IStatefulResponseHandler<ResultType, DeserializedType> {
      * @throws Exception an exception occurs if the request was unable to complete for any reason
      */
     @Nullable
+    @SuppressFBWarnings
     <ResponseType> ResultType generateResult(@Nonnull final IHttpRequest request,
                               @Nonnull final ResponseType response,
                               @Nonnull final ISerializer serializer,
diff --git a/src/main/java/com/microsoft/graph/tasks/LargeFileUploadResponseHandler.java b/src/main/java/com/microsoft/graph/tasks/LargeFileUploadResponseHandler.java
@@ -40,6 +40,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import okhttp3.MediaType;
 import okhttp3.Response;
 import okhttp3.ResponseBody;
@@ -72,6 +73,7 @@ protected LargeFileUploadResponseHandler(@Nonnull final Class<UploadType> upload
 
 	@Override
 	@Nullable
+    @SuppressFBWarnings
 	public <ResponseType> LargeFileUploadResponse<UploadType> generateResult(
 			@Nonnull final IHttpRequest request,
 			@Nonnull final ResponseType response,
diff --git a/src/main/java/com/microsoft/graph/tasks/LargeFileUploadTask.java b/src/main/java/com/microsoft/graph/tasks/LargeFileUploadTask.java
@@ -25,6 +25,7 @@
 import com.microsoft.graph.core.ClientException;
 import com.microsoft.graph.core.IBaseClient;
 import com.microsoft.graph.options.Option;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 import java.io.IOException;
 import java.io.InputStream;
@@ -99,6 +100,7 @@ public class LargeFileUploadTask<UploadType> {
      * @param streamSize      the stream size
      * @param uploadTypeClass the upload type class
      */
+    @SuppressFBWarnings
     public LargeFileUploadTask(@Nonnull final IUploadSession uploadSession,
                                  @Nonnull final IBaseClient<?> client,
                                  @Nonnull final InputStream inputStream,
diff --git a/src/test/java/com/microsoft/graph/authentication/TokenCredentialAuthProviderTest.java b/src/test/java/com/microsoft/graph/authentication/TokenCredentialAuthProviderTest.java
@@ -3,6 +3,7 @@
 import com.azure.core.credential.TokenCredential;
 import com.microsoft.graph.mocks.MockTokenCredential;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.*;
@@ -13,6 +14,7 @@
 import java.util.ArrayList;
 import java.util.concurrent.ExecutionException;
 
+@SuppressFBWarnings
 public class TokenCredentialAuthProviderTest {
 
     private static final String testToken = "CredentialTestToken";
diff --git a/src/test/java/com/microsoft/graph/content/BatchRequestContentTest.java b/src/test/java/com/microsoft/graph/content/BatchRequestContentTest.java
@@ -29,6 +29,7 @@
 import com.microsoft.graph.serializer.IJsonBackedObject;
 import com.microsoft.graph.serializer.ISerializer;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.ValueSource;
@@ -41,6 +42,7 @@
 import okhttp3.Response;
 import okhttp3.ResponseBody;
 
+@SuppressFBWarnings
 class BatchRequestContentTest {
 
     String testurl = "http://graph.microsoft.com/v1.0/me";
diff --git a/src/test/java/com/microsoft/graph/content/BatchResponseContentTest.java b/src/test/java/com/microsoft/graph/content/BatchResponseContentTest.java
@@ -20,8 +20,10 @@
 import com.microsoft.graph.serializer.DefaultSerializer;
 import com.microsoft.graph.serializer.ISerializer;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 
+@SuppressFBWarnings
 public class BatchResponseContentTest {
     @Test
     public void testValidBatchResponseContent() {
diff --git a/src/test/java/com/microsoft/graph/core/BaseClientTests.java b/src/test/java/com/microsoft/graph/core/BaseClientTests.java
@@ -38,7 +38,7 @@ public class BaseClientTests {
     private ISerializer mSerializer;
 
 	@BeforeEach
-	public void setUp() throws Exception {
+	public void setUp() {
         baseClient = new BaseClient<>();
         mLogger = mock(ILogger.class);
         mSerializer = mock(ISerializer.class);
diff --git a/src/test/java/com/microsoft/graph/core/ClientExceptionTests.java b/src/test/java/com/microsoft/graph/core/ClientExceptionTests.java
@@ -12,7 +12,7 @@ public class ClientExceptionTests {
     private String expectMessage = "This is test exception message";
 
 	@BeforeEach
-	public void setUp() throws Exception {
+	public void setUp() {
         clientException = new ClientException(expectMessage, null);
 	}
 
diff --git a/src/test/java/com/microsoft/graph/core/GraphServiceClientTest.java b/src/test/java/com/microsoft/graph/core/GraphServiceClientTest.java
@@ -12,6 +12,7 @@
 import java.util.Map;
 import java.util.concurrent.CompletableFuture;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 
 import okhttp3.Request;
@@ -28,6 +29,7 @@
 
 import javax.annotation.Nullable;
 
+@SuppressFBWarnings
 public class GraphServiceClientTest {
     private IAuthenticationProvider getAuthProvider() {
         return mock(IAuthenticationProvider.class);
diff --git a/src/test/java/com/microsoft/graph/http/BaseCollectionPageTests.java b/src/test/java/com/microsoft/graph/http/BaseCollectionPageTests.java
@@ -26,7 +26,7 @@ public class BaseCollectionPageTests {
 
     @BeforeEach
     @SuppressWarnings("unchecked")
-    public void setUp() throws Exception {
+    public void setUp() {
         list = new ArrayList<String>();
         list.add("Object1");
         list.add("Object2");
diff --git a/src/test/java/com/microsoft/graph/http/BaseCollectionRequestTests.java b/src/test/java/com/microsoft/graph/http/BaseCollectionRequestTests.java
@@ -10,6 +10,7 @@
 import com.microsoft.graph.options.QueryOption;
 import com.microsoft.graph.serializer.ISerializer;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -56,6 +57,7 @@ public class BaseCollectionRequestTests {
     private BaseEntityCollectionRequest<JsonObject, ICollectionResponse<JsonObject>, BaseCollectionPage<JsonObject, BaseRequestBuilder<JsonObject>>> mRequest;
 
     @BeforeEach
+    @SuppressFBWarnings
     @SuppressWarnings("unchecked")
     public void setUp() throws Exception {
         mBaseClient = mock(IBaseClient.class);
diff --git a/src/test/java/com/microsoft/graph/http/BaseRequestBuilderTests.java b/src/test/java/com/microsoft/graph/http/BaseRequestBuilderTests.java
@@ -18,7 +18,7 @@ public class BaseRequestBuilderTests {
     private BaseRequestBuilder<String> baseRequestBuilder;
 
     @BeforeEach
-    public void setUp() throws Exception {
+    public void setUp() {
         baseRequestBuilder = new BaseRequestBuilder<>(expectedRequestUrl, mock(IBaseClient.class), null){};
     }
 
diff --git a/src/test/java/com/microsoft/graph/http/BaseRequestTests.java b/src/test/java/com/microsoft/graph/http/BaseRequestTests.java
@@ -11,6 +11,7 @@
 
 import static org.mockito.Mockito.mock;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -37,6 +38,7 @@ public class BaseRequestTests {
     private BaseRequest<JsonObject> mRequest;
 
     @BeforeEach
+    @SuppressFBWarnings
     public void setUp() throws Exception {
         final Response response = new Response.Builder()
                 .request(new Request.Builder().url("https://a.b.c").build())
diff --git a/src/test/java/com/microsoft/graph/http/BaseStreamRequestTests.java b/src/test/java/com/microsoft/graph/http/BaseStreamRequestTests.java
@@ -10,6 +10,7 @@
 import java.io.InputStream;
 import java.util.concurrent.ExecutionException;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.mockito.stubbing.Answer;
@@ -33,13 +34,14 @@
 /**
  * Test cases for {@see BaseStreamRequest}
  */
+@SuppressFBWarnings
 public class BaseStreamRequestTests {
 
     private BaseClient<Request> mBaseClient;
 
     @BeforeEach
     @SuppressWarnings("unchecked")
-    public void setUp() throws Exception {
+    public void setUp() {
         mBaseClient = mock(BaseClient.class);
     }
 
diff --git a/src/test/java/com/microsoft/graph/http/CoreHttpCallbackFutureWrapperTests.java b/src/test/java/com/microsoft/graph/http/CoreHttpCallbackFutureWrapperTests.java
@@ -9,11 +9,13 @@
 import java.io.IOException;
 import java.util.concurrent.ExecutionException;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 
 import okhttp3.Call;
 import okhttp3.Response;
 
+@SuppressFBWarnings
 class CoreHttpCallbackFutureWrapperTests {
 
     @Test
diff --git a/src/test/java/com/microsoft/graph/http/CoreHttpProviderTests.java b/src/test/java/com/microsoft/graph/http/CoreHttpProviderTests.java
@@ -20,6 +20,7 @@
 import com.microsoft.graph.serializer.DefaultSerializer;
 import com.microsoft.graph.serializer.ISerializer;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import okio.Buffer;
 import org.junit.jupiter.api.Test;
 
@@ -62,6 +63,7 @@ class CoreHttpProviderTests {
     private Gson GSON = new GsonBuilder().create();
 
     @Test
+    @SuppressFBWarnings
     void testErrorResponse() throws Exception {
         final GraphErrorCodes expectedErrorCode = GraphErrorCodes.INVALID_REQUEST;
         final String expectedMessage = "Test error!";
@@ -85,6 +87,7 @@ void testErrorResponse() throws Exception {
     }
 
     @Test
+    @SuppressFBWarnings
     void testVerboseErrorResponse() throws Exception {
         final GraphErrorCodes expectedErrorCode = GraphErrorCodes.INVALID_REQUEST;
         final String expectedMessage = "Test error!";
diff --git a/src/test/java/com/microsoft/graph/httpcore/AuthenticationHandlerTest.java b/src/test/java/com/microsoft/graph/httpcore/AuthenticationHandlerTest.java
@@ -10,13 +10,15 @@
 import java.net.URL;
 import java.util.concurrent.CompletableFuture;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 
 import okhttp3.OkHttpClient;
 import okhttp3.Request;
 
 public class AuthenticationHandlerTest {
     @Test
+    @SuppressFBWarnings
     public void testAuthenticationHandler() throws Exception {
         IAuthenticationProvider authProvider = mock(IAuthenticationProvider.class);
         when(authProvider.getAuthorizationTokenAsync(any(URL.class))).thenReturn(CompletableFuture.completedFuture("a token"));
diff --git a/src/test/java/com/microsoft/graph/httpcore/RetryHandlerTest.java b/src/test/java/com/microsoft/graph/httpcore/RetryHandlerTest.java
@@ -9,6 +9,7 @@
 import java.io.IOException;
 import java.net.HttpURLConnection;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.junit.jupiter.api.Test;
 
 import com.microsoft.graph.httpcore.middlewareoption.IShouldRetry;
@@ -21,6 +22,7 @@
 import okhttp3.Response;
 import okio.BufferedSink;
 
+@SuppressFBWarnings
 public class RetryHandlerTest {
 
     private static final String testmeurl = "https://graph.microsoft.com/v1.0/me";
diff --git a/src/test/java/com/microsoft/graph/httpcore/middlewareoption/RedirectOptionsTest.java b/src/test/java/com/microsoft/graph/httpcore/middlewareoption/RedirectOptionsTest.java
diff --git a/src/test/java/com/microsoft/graph/httpcore/middlewareoption/RetryOptionsTest.java b/src/test/java/com/microsoft/graph/httpcore/middlewareoption/RetryOptionsTest.java
diff --git a/src/test/java/com/microsoft/graph/options/OptionTests.java b/src/test/java/com/microsoft/graph/options/OptionTests.java
diff --git a/src/test/java/com/microsoft/graph/serializer/ByteArraySerializerTests.java b/src/test/java/com/microsoft/graph/serializer/ByteArraySerializerTests.java
diff --git a/src/test/java/com/microsoft/graph/serializer/DateOnlyTests.java b/src/test/java/com/microsoft/graph/serializer/DateOnlyTests.java
diff --git a/src/test/java/com/microsoft/graph/serializer/DurationTests.java b/src/test/java/com/microsoft/graph/serializer/DurationTests.java
diff --git a/src/test/java/com/microsoft/graph/serializer/EdmNativeTypeSerializerTests.java b/src/test/java/com/microsoft/graph/serializer/EdmNativeTypeSerializerTests.java
diff --git a/src/test/java/com/microsoft/graph/serializer/ISO8601Test.java b/src/test/java/com/microsoft/graph/serializer/ISO8601Test.java
diff --git a/src/test/java/com/microsoft/graph/serializer/OffsetDateTimeSerializerTests.java b/src/test/java/com/microsoft/graph/serializer/OffsetDateTimeSerializerTests.java
diff --git a/src/test/java/com/microsoft/graph/serializer/TimeOfDayTests.java b/src/test/java/com/microsoft/graph/serializer/TimeOfDayTests.java

Original file line number	Diff line number	Diff line change
`@@ -8,8 +8,8 @@ buildscript {`
`8`	`8`	`}`
`9`	`9`
`10`	`10`	`dependencies {`
`11`		`- classpath "com.gradle:gradle-enterprise-gradle-plugin:3.10"`
`12`		`- classpath "com.android.tools.build:gradle:7.1.3"`
	`11`	`+ classpath "com.gradle:gradle-enterprise-gradle-plugin:3.10.1"`
	`12`	`+ classpath "com.android.tools.build:gradle:7.2.1"`
`13`	`13`	`classpath "com.github.ben-manes:gradle-versions-plugin:0.42.0"`
`14`	`14`	`}`
`15`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ plugins {`
`14`	`14`	`id 'maven-publish'`
`15`	`15`	`id 'signing'`
`16`	`16`	`id 'jacoco'`
`17`		`- id 'com.github.spotbugs' version '5.0.6'`
	`17`	`+ id 'com.github.spotbugs' version '5.0.7'`
`18`	`18`	`id "org.sonarqube" version "3.3"`
`19`	`19`	`}`
`20`	`20`
Original file line number	Diff line number	Diff line change
`@@ -12,5 +12,5 @@ repositories {`
`12`	`12`	`dependencies {`
`13`	`13`	`testImplementation group: 'junit', name: 'junit', version: '4.13.2'`
`14`	`14`	`implementation project(':coreLibrary')`
`15`		`- implementation 'com.azure:azure-identity:1.5.0'`
	`15`	`+ implementation 'com.azure:azure-identity:1.5.1'`
`16`	`16`	`}`