MongoConfig - Removed Sonar warnings 19/109419/1
authorChris André <chris.andre@yoppworks.com>
Mon, 22 Jun 2020 15:15:43 +0000 (11:15 -0400)
committerChris André <chris.andre@yoppworks.com>
Mon, 22 Jun 2020 15:15:43 +0000 (11:15 -0400)
- updated pom.xml files to get rid of warnings
- used `try-with-resource` to automatically close variable `mongoC`

Issue-ID: AAI-2954
Signed-off-by: Chris Andre <chris.andre@yoppworks.com>
Change-Id: Ib26db5f100d700c2e18d7aa2e5376dbe17f7f846

pom.xml
src/main/java/org/onap/aai/cacher/common/MongoConfig.java

diff --git a/pom.xml b/pom.xml
index efbf33e..04eb0f6 100644 (file)
--- a/pom.xml
+++ b/pom.xml
                        <artifactId>commons-compress</artifactId>
                        <version>1.18</version>
                </dependency>
-               <dependency>
-                       <groupId>org.apache.commons</groupId>
-                       <artifactId>commons-compress</artifactId>
-                       <version>1.18</version>
-               </dependency>
                <dependency>
                        <groupId>org.mongodb</groupId>
                        <artifactId>mongodb-driver</artifactId>
                        <artifactId>jersey-common</artifactId>
                        <version>2.24</version>
                </dependency>
-               <dependency>
-                       <groupId>org.json</groupId>
-                       <artifactId>json</artifactId>
-               </dependency>
                <dependency>
                        <groupId>com.googlecode.json-simple</groupId>
                        <artifactId>json-simple</artifactId>
                        <plugin>
                                <artifactId>maven-dependency-plugin</artifactId>
                        </plugin>
-                       <plugin>
-                               <groupId>org.springframework.boot</groupId>
-                               <artifactId>spring-boot-maven-plugin</artifactId>
-                       </plugin>
                        <plugin>
                                <artifactId>exec-maven-plugin</artifactId>
                                <groupId>org.codehaus.mojo</groupId>
+                               <version>3.0.0</version>
                        </plugin>
                        <plugin>
                                <groupId>org.springframework.boot</groupId>
                        <plugin>
                                <groupId>com.github.kongchen</groupId>
                                <artifactId>swagger-maven-plugin</artifactId>
+                               <version>3.1.8</version>
                        </plugin>
                        <plugin>
                                <groupId>org.apache.maven.plugins</groupId>
                                <artifactId>maven-javadoc-plugin</artifactId>
+                               <version>3.2.0</version>
                                <configuration>
                                        <additionalparam>-Xdoclint:none</additionalparam>
                                </configuration>
index 5c1a541..ca063a7 100644 (file)
@@ -48,7 +48,7 @@ import java.io.IOException;
 @Configuration
 public class MongoConfig {
 
-    private final static EELFLogger EELF_LOGGER = EELFManager.getInstance().getLogger(MongoConfig.class);
+    private static final EELFLogger EELF_LOGGER = EELFManager.getInstance().getLogger(MongoConfig.class);
 
     @Value("${mongodb.host}")
     private String MONGO_DB_HOST;
@@ -61,9 +61,8 @@ public class MongoConfig {
 
     @Bean
     public MongoClient mongoClient(MongodProcess mongodProcess) {
-        try {
-            // To connect to mongodb server
-            MongoClient mongoC = new MongoClient(MONGO_DB_HOST, MONGO_DB_PORT);
+        // To connect to mongodb server
+        try (MongoClient mongoC = new MongoClient(MONGO_DB_HOST, MONGO_DB_PORT)) {
 
             // Now connect to your databases
             EELF_LOGGER.info("Connect to database successfully");