Ensure httpVersion is now always defined in exposed data

Author: pimterry

src/rules/requests/request-handlers.ts

@@ -932,6 +932,7 @@ export class PassThroughHandler extends PassThroughHandlerDefinition {
                         id: clientReq.id,
                         protocol: protocol?.replace(':', '') ?? '',
                         method: method,
+                        httpVersion: serverRes.httpVersion,
                         url: reqUrl,
                         destination: {
                             hostname: hostname || 'localhost',

src/server/mockttp-server.ts

@@ -1069,7 +1069,7 @@ ${await this.suggestRule(request)}`
 
             const request: ClientError['request'] = {
                 ...commonParams,
-                httpVersion: parsedRequest.httpVersion,
+                httpVersion: parsedRequest.httpVersion || '1.1',
                 method: parsedRequest.method,
                 protocol: parsedRequest.protocol,
                 url: parsedRequest.url,

src/types.ts

@@ -59,7 +59,7 @@ export interface Request {
     matchedRuleId?: string;
 
     protocol: string;
-    httpVersion?: string; // Like timingEvents - not set remotely with older servers
+    httpVersion: string;
     method: string;
     url: string;
     path: string;
@@ -434,7 +434,7 @@ export interface ClientError {
 
         // All of these are best guess, depending on what's parseable:
         protocol?: string;
-        httpVersion?: string;
+        httpVersion: string;
         method?: string;
         url?: string;
         path?: string;