docs: update CHANGELOG.md

bednar · bednar · commit 141e084c9fbe · 2021-11-10T08:41:58.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,7 +7,10 @@
 
 ### Features
 1. [#272](https://github.com/influxdata/influxdb-client-java/pull/272): Add `PingService` to check status of OSS and Cloud instance
-2. [#278](https://github.com/influxdata/influxdb-client-java/pull/278): Add query method with all params for BucketsApi, OrganizationApi and TasksApi
+1. [#278](https://github.com/influxdata/influxdb-client-java/pull/278): Add query method with all params for BucketsApi, OrganizationApi and TasksApi
+
+### Bug Fixes
+1. [#279](https://github.com/influxdata/influxdb-client-java/pull/279): Session authentication for InfluxDB `2.1`
 
 ### CI
 1. [#275](https://github.com/influxdata/influxdb-client-java/pull/275): Deploy `influxdb-client-test` package into Maven repository
diff --git a/client/src/main/java/com/influxdb/client/internal/AuthenticateInterceptor.java b/client/src/main/java/com/influxdb/client/internal/AuthenticateInterceptor.java
@@ -147,16 +147,18 @@ void signout() throws IOException {
             return;
         }
 
-        Request authRequest = new Request.Builder()
+        Request.Builder authRequest = new Request.Builder()
                 .url(buildPath("api/v2/signout"))
-                .post(RequestBody.create("application/json", null))
-                .header("Cookie", string(sessionCookies))
-                .build();
+                .post(RequestBody.create("application/json", null));
 
-        this.signout.set(true);
-        this.sessionCookies = null;
+        if (sessionCookies != null) {
+            authRequest.addHeader("Cookie", string(sessionCookies));
+        }
+
+        signout.set(true);
+        sessionCookies = null;
 
-        Response response = this.okHttpClient.newCall(authRequest).execute();
+        Response response = okHttpClient.newCall(authRequest.build()).execute();
         response.close();
     }
 
diff --git a/client/src/test/java/com/influxdb/client/internal/AuthenticateInterceptorTest.java b/client/src/test/java/com/influxdb/client/internal/AuthenticateInterceptorTest.java
@@ -117,7 +117,7 @@ void authorizationSession() throws IOException, InterruptedException {
         RecordedRequest requestToTasks = mockServer.takeRequest();
         Assertions.assertThat(requestToTasks.getPath()).endsWith("/api/v2/tasks");
         Assertions.assertThat(requestToTasks.getHeader("Cookie"))
-                .isEqualTo("session=yCgXaEBF8mYSmJUweRcW0g_5jElMs7mv6_-G1bNcau4Z0ZLQYtj0BkHZYRnBVA6uXHtyuhflcOzyNDNRxnaC0A==");
+                .isEqualTo("session=yCgXaEBF8mYSmJUweRcW0g_5jElMs7mv6_-G1bNcau4Z0ZLQYtj0BkHZYRnBVA6uXHtyuhflcOzyNDNRxnaC0A==; path=/api/v2");
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ void authorizationSession() throws IOException, InterruptedException {`
`117`	`117`	`RecordedRequest requestToTasks = mockServer.takeRequest();`
`118`	`118`	`Assertions.assertThat(requestToTasks.getPath()).endsWith("/api/v2/tasks");`
`119`	`119`	`Assertions.assertThat(requestToTasks.getHeader("Cookie"))`
`120`		`- .isEqualTo("session=yCgXaEBF8mYSmJUweRcW0g_5jElMs7mv6_-G1bNcau4Z0ZLQYtj0BkHZYRnBVA6uXHtyuhflcOzyNDNRxnaC0A==");`
	`120`	`+ .isEqualTo("session=yCgXaEBF8mYSmJUweRcW0g_5jElMs7mv6_-G1bNcau4Z0ZLQYtj0BkHZYRnBVA6uXHtyuhflcOzyNDNRxnaC0A==; path=/api/v2");`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`@Test`