Merge pull request #1234 from dmetzgar/fix-tracecontextcorrelation

trask · web-flow · commit 67ff6bec6fbe · 2020-05-18T16:58:07.000-07:00
Fix index out of bounds exception
diff --git a/web/src/main/java/com/microsoft/applicationinsights/web/internal/correlation/TraceContextCorrelation.java b/web/src/main/java/com/microsoft/applicationinsights/web/internal/correlation/TraceContextCorrelation.java
@@ -370,7 +370,7 @@ public static String generateChildDependencyTarget(String requestContext) {
         String[] keyValue = requestContext.split("=");
 
         String headerAppID = null;
-        if (keyValue[0].equals(REQUEST_CONTEXT_HEADER_APPID_KEY)) {
+        if (keyValue.length == 2 && keyValue[0].equals(REQUEST_CONTEXT_HEADER_APPID_KEY)) {
             headerAppID = keyValue[1];
         }
 

Original file line number	Diff line number	Diff line change
`@@ -370,7 +370,7 @@ public static String generateChildDependencyTarget(String requestContext) {`
`370`	`370`	`String[] keyValue = requestContext.split("=");`
`371`	`371`
`372`	`372`	`String headerAppID = null;`
`373`		`- if (keyValue[0].equals(REQUEST_CONTEXT_HEADER_APPID_KEY)) {`
	`373`	`+ if (keyValue.length == 2 && keyValue[0].equals(REQUEST_CONTEXT_HEADER_APPID_KEY)) {`
`374`	`374`	`headerAppID = keyValue[1];`
`375`	`375`	`}`
`376`	`376`