diff options
author | VENKATESH KUMAR <vv770d@att.com> | 2018-02-28 14:12:50 -0500 |
---|---|---|
committer | VENKATESH KUMAR <vv770d@att.com> | 2018-02-28 14:13:14 -0500 |
commit | 4db8e80971cba267628c6b642aed902eabbde5d8 (patch) | |
tree | b47ecba86ea22046b7bc37762a564a3eb197af40 | |
parent | e6cb4531f2dfd8c5e4e3b90c69891cca686142ee (diff) |
fix CLM issue + logging updates
Change-Id: I73a1fdd0dc0624332fc799761e8681af7f19dd3c
Signed-off-by: VENKATESH KUMAR <vv770d@att.com>
Issue-ID: DCAEGEN2-360
-rw-r--r-- | pom.xml | 59 | ||||
-rw-r--r-- | src/main/java/org/onap/dcae/commonFunction/CommonStartup.java | 9 | ||||
-rw-r--r-- | src/main/java/org/onap/dcae/restapi/endpoints/EventReceipt.java | 5 |
3 files changed, 49 insertions, 24 deletions
@@ -22,10 +22,10 @@ ECOMP is a trademark and service mark of AT&T Intellectual Property. <modelVersion>4.0.0</modelVersion> <parent> - <groupId>org.onap.oparent</groupId> - <artifactId>oparent</artifactId> - <version>0.1.1</version> - <relativePath/> + <groupId>org.onap.oparent</groupId> + <artifactId>oparent</artifactId> + <version>0.1.1</version> + <relativePath /> </parent> <groupId>org.onap.dcaegen2.collectors.ves</groupId> <artifactId>VESCollector</artifactId> @@ -82,9 +82,15 @@ ECOMP is a trademark and service mark of AT&T Intellectual Property. </dependency> <dependency> + <groupId>com.google.guava</groupId> + <artifactId>guava</artifactId> + <version>18.0</version> + </dependency> + + <dependency> <groupId>com.github.fge</groupId> <artifactId>json-schema-validator</artifactId> - <version>2.0.1</version> + <version>2.2.6</version> </dependency> <dependency> @@ -122,6 +128,9 @@ ECOMP is a trademark and service mark of AT&T Intellectual Property. <version>20160810</version> </dependency> + <!-- https://mvnrepository.com/artifact/com.github.fge/jackson-coreutils --> + + <dependency> <groupId>commons-configuration</groupId> <artifactId>commons-configuration</artifactId> @@ -273,7 +282,7 @@ ECOMP is a trademark and service mark of AT&T Intellectual Property. <artifactId>maven-jar-plugin</artifactId> </plugin> - + <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-source-plugin</artifactId> @@ -308,18 +317,18 @@ ECOMP is a trademark and service mark of AT&T Intellectual Property. </configuration> </execution> <execution> - <id>install script</id> - <phase>install</phase> - <goals> - <goal>exec</goal> - </goals> - <configuration> - <arguments> - <argument>__</argument> - <argument>install</argument> - </arguments> - </configuration> - </execution> + <id>install script</id> + <phase>install</phase> + <goals> + <goal>exec</goal> + </goals> + <configuration> + <arguments> + <argument>__</argument> + <argument>install</argument> + </arguments> + </configuration> + </execution> </executions> </plugin> @@ -364,5 +373,19 @@ ECOMP is a trademark and service mark of AT&T Intellectual Property. </reporting> + <dependencyManagement> + <dependencies> + <dependency> + <groupId>com.github.fge</groupId> + <artifactId>json-schema-core</artifactId> + <version>1.2.5</version> + </dependency> + <dependency> + <groupId>javax.mail</groupId> + <artifactId>mail</artifactId> + <version>1.4.7</version> + </dependency> + </dependencies> + </dependencyManagement> </project> diff --git a/src/main/java/org/onap/dcae/commonFunction/CommonStartup.java b/src/main/java/org/onap/dcae/commonFunction/CommonStartup.java index 6b5502f7..83c81722 100644 --- a/src/main/java/org/onap/dcae/commonFunction/CommonStartup.java +++ b/src/main/java/org/onap/dcae/commonFunction/CommonStartup.java @@ -34,12 +34,13 @@ import com.att.nsa.drumlin.till.nv.rrNvReadable.loadException; import com.att.nsa.drumlin.till.nv.rrNvReadable.missingReqdSetting;
import com.fasterxml.jackson.core.JsonParseException;
import com.fasterxml.jackson.databind.JsonNode;
-import com.github.fge.jsonschema.exceptions.ProcessingException;
+import com.github.fge.jackson.JsonLoader;
+import com.github.fge.jsonschema.core.exceptions.ProcessingException;
+import com.github.fge.jsonschema.core.report.ProcessingMessage;
+import com.github.fge.jsonschema.core.report.ProcessingReport;
import com.github.fge.jsonschema.main.JsonSchema;
import com.github.fge.jsonschema.main.JsonSchemaFactory;
-import com.github.fge.jsonschema.report.ProcessingMessage;
-import com.github.fge.jsonschema.report.ProcessingReport;
-import com.github.fge.jsonschema.util.JsonLoader;
+
import org.apache.catalina.LifecycleException;
import org.json.JSONArray;
import org.json.JSONException;
diff --git a/src/main/java/org/onap/dcae/restapi/endpoints/EventReceipt.java b/src/main/java/org/onap/dcae/restapi/endpoints/EventReceipt.java index 204e5058..f5a5e6d8 100644 --- a/src/main/java/org/onap/dcae/restapi/endpoints/EventReceipt.java +++ b/src/main/java/org/onap/dcae/restapi/endpoints/EventReceipt.java @@ -113,7 +113,8 @@ public class EventReceipt extends NsaBaseEndpoint { retkey = NsaBaseEndpoint.getAuthenticatedUser(ctx);
}
} catch (NullPointerException x) {
- log.info("Invalid user request " + userId + ctx.request().getContentType() + MESSAGE + jsonObject);
+ //log.info("Invalid user request :" + userId + " FROM " + ctx.request().getRemoteAddress() + " " + ctx.request().getContentType() + MESSAGE + jsonObject);
+ log.info(String.format("Unauthorized request %s FROM %s %s %s %s", getUser(ctx), ctx.request().getRemoteAddress(), ctx.request().getContentType(), MESSAGE, jsonObject));
CommonStartup.eplog.info("EVENT_RECEIPT_FAILURE: Unauthorized user" + userId + x);
respondWithCustomMsginJson(ctx, HttpStatusCodes.k401_unauthorized, "Invalid user");
return;
@@ -232,7 +233,7 @@ public class EventReceipt extends NsaBaseEndpoint { CommonStartup.handleEvents(jsonArrayMod);
} else {
- log.info(String.format("Unauthorized request %s%s%s%s", getUser(ctx), ctx.request().getContentType(), MESSAGE,
+ log.info(String.format("Unauthorized request %s FROM %s %s %s %s", getUser(ctx), ctx.request().getRemoteAddress(), ctx.request().getContentType(), MESSAGE,
jsonObject));
respondWithCustomMsginJson(ctx, HttpStatusCodes.k401_unauthorized, "Unauthorized user");
ErrorStatus=true;
|