set connectionTimeZone default to SERVER (#89)

* Set default `connectionTimeZone` parameter for MYSQL to `SERVER`

Author: if0s

CHANGELOG.md

@@ -1,3 +1,6 @@
+## 2.5.5 (May 20, 2022)
+* Set default `connectionTimeZone` parameter for MYSQL to `SERVER`
+
 ## 2.5.4 (May 11, 2022)
 * Made an automated vulnerability check run in CI/CD
 

build.gradle

@@ -72,7 +72,7 @@ dependencies {
     compile 'com.fasterxml.jackson.core:jackson-core:2.10.1'
     compile 'com.fasterxml.jackson.core:jackson-annotations:2.10.1'
     compile 'com.fasterxml.jackson.core:jackson-databind:2.13.2.2'
-    compile 'com.google.code.gson:gson:2.8.6'
+    compile 'com.google.code.gson:gson:2.9.0'
     compile 'com.microsoft.sqlserver:mssql-jdbc:6.4.0.jre8'
     compile 'io.elastic:sailor-jvm:3.3.9'
     compile 'mysql:mysql-connector-java:8.0.29'

component.json

@@ -1,7 +1,7 @@
 {
   "title": "Database",
   "description": "Database JDBC connector",
-  "version": "2.5.4",
+  "version": "2.5.5",
   "credentials": {
     "verifier": "io.elastic.jdbc.JdbcCredentialsVerifier",
     "fields": {

src/main/java/io/elastic/jdbc/utils/Utils.java

@@ -101,6 +101,9 @@ private static Properties getConfigurationProperties(final JsonObject config,
         throw new RuntimeException(e);
       }
     }
+    if (properties.getProperty("connectionTimeZone") == null && engineType == Engines.MYSQL) {
+      properties.setProperty("connectionTimeZone", "SERVER");
+    }
     properties.setProperty("user", user);
     properties.setProperty("password", password);
     return properties;

src/test/groovy/io/elastic/jdbc/unit/UtilsSpec.groovy

@@ -16,6 +16,7 @@ class UtilsSpec extends Specification {
         .add("password", "password")
         .build();
     Properties expectedProperties = new Properties();
+    expectedProperties.setProperty("connectionTimeZone", "SERVER")
     expectedProperties.setProperty("user", "user")
     expectedProperties.setProperty("password", "password")
     Properties properties = Utils.getConfigurationProperties(config, engineType)
@@ -31,6 +32,7 @@ class UtilsSpec extends Specification {
         .add("configurationProperties", "ssl=true")
         .build();
     Properties expectedProperties = new Properties();
+    expectedProperties.setProperty("connectionTimeZone", "SERVER")
     expectedProperties.setProperty("user", "user")
     expectedProperties.setProperty("password", "password")
     expectedProperties.setProperty("ssl", "true")
@@ -47,6 +49,7 @@ class UtilsSpec extends Specification {
         .add("configurationProperties", "ssl=true&serverTimezone=UTC")
         .build();
     Properties expectedProperties = new Properties();
+    expectedProperties.setProperty("connectionTimeZone", "SERVER")
     expectedProperties.setProperty("user", "user")
     expectedProperties.setProperty("password", "password")
     expectedProperties.setProperty("ssl", "true")