Merge pull request #335 from newrelic/feature/runtime-error/add-uri

Added request URI to application runtime error event, enhancing error logging and debugging capabilities.

newrelic-security-api/src/main/java/com/newrelic/api/agent/security/schema/HttpRequest.java

@@ -10,6 +10,8 @@ public class HttpRequest {
 
     @JsonIgnore
     public static final int MAX_ALLOWED_REQUEST_BODY_LENGTH = 500000;
+    @JsonIgnore
+    public static final String QUESTION_MARK = "?";
 
     private StringBuilder body;
 
@@ -40,6 +42,7 @@ public class HttpRequest {
     private boolean isGrpc;
 
     private String route;
+    private String requestURI;
 
     private Map<String, String> customDataType;
 
@@ -73,6 +76,7 @@ public HttpRequest() {
         this.isRequestParsed = false;
         this.isGrpc = false;
         this.route = StringUtils.EMPTY;
+        this.requestURI = StringUtils.EMPTY;
         this.customDataType = new HashMap<>();
     }
 
@@ -91,6 +95,7 @@ public HttpRequest(HttpRequest servletInfo) {
         this.isRequestParsed = servletInfo.isRequestParsed;
         this.isGrpc = servletInfo.isGrpc;
         this.route = servletInfo.route;
+        this.requestURI = servletInfo.requestURI;
         this.pathParameterMap = servletInfo.pathParameterMap;
         this.queryParameters = servletInfo.queryParameters;
         this.requestHeaderParameters = servletInfo.requestHeaderParameters;
@@ -113,6 +118,7 @@ public String getUrl() {
 
     public void setUrl(String url) {
         this.url = url;
+        this.requestURI = StringUtils.substringBefore(url, QUESTION_MARK);
     }
 
     public Map<String, String> getHeaders() {
@@ -249,6 +255,14 @@ public void setRoute(String route){
         this.route = StringUtils.removeEnd(StringUtils.prependIfMissing(route, StringUtils.SEPARATOR), StringUtils.SEPARATOR);
     }
 
+    public String getRequestURI() {
+        return requestURI;
+    }
+
+    public void setRequestURI(String requestURI) {
+        this.requestURI = requestURI;
+    }
+
     public void setRoute(String segment, boolean isAlreadyServlet) {
         // remove servlet detected route if another framework detected;
         if (isAlreadyServlet) {