Fix request line not being included in request headers

nccgroup · Dec 5, 2023 · bc8a894 · bc8a894
1 parent b630d53
commit bc8a894
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/main/java/com/nccgroup/loggerplusplus/logentry/LogEntry.java b/src/main/java/com/nccgroup/loggerplusplus/logentry/LogEntry.java
@@ -177,7 +177,7 @@ public Status getPreviousStatus() {
  private Status processRequest() {
 
 
- requestHeaders = request.headers();
+ requestHeaders = new ArrayList<>(request.headers());
 
  this.requestHttpVersion = request.httpVersion();
 
@@ -513,8 +513,11 @@ public Object getValueByKey(LogEntryField columnName) {
  return response.body().length();
  case RTT:
  return requestResponseDelay;
- case REQUEST_HEADERS:
- return requestHeaders != null ? requestHeaders.stream().map(HttpHeader::toString).collect(Collectors.joining("\r\n")) : "";
+ case REQUEST_HEADERS: {
+ if(requestHeaders == null) return "";
+ //Hacky workaround since Burp doesn't include path in headers.
+ return String.format("%s %s %s\r\n%s", request.method(), request.path(), request.httpVersion(), requestHeaders.stream().map(HttpHeader::toString).collect(Collectors.joining("\r\n")));
+ }
  case RESPONSE_HEADERS:
  return responseHeaders != null ? responseHeaders.stream().map(HttpHeader::toString).collect(Collectors.joining("\r\n")) : "";
  case REDIRECT_URL: