@uoa/lambda-tracing 2.1.0 → 2.1.2

package/changelog.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 2.1.2
+- Updated XML response parsing to include attribute tags
+
+## 2.1.1
+- Fixed issue where JSON requests & responses could only have content-type application/json. Will now parse request/response if content-type header includes "json" anywhere. If "json" is not present, it will check for "xml", then "image". If none are present, the raw string body will be used.
+
 ## 2.1.0
 - Unexpected errors are now caught within the library. When this occurs, the request promise will reject with the error.
 - http response promises will now be rejected if the response code is not in the 2xx range

package/dist/uoaHttps.js

@@ -111,17 +111,18 @@ function doHttpsRequest(options, resolve, reject) {
                     let parsedBody = body.toString();
                     //Get the response content-type header value so we can apply different parsing methods
                     const responseContentType = response.headers["content-type"]?.toLowerCase() ?? '';
-                    if (responseContentType === '' || responseContentType.includes('application/json')) {
+                    if (responseContentType === '' || responseContentType.includes('json')) {
                         parsedBody = JSON.parse(body.toString());
                     }
-                    else if (responseContentType.includes('application/xml') || responseContentType.includes('text/xml')) {
+                    else if (responseContentType.includes('xml')) {
                         const parser = new fast_xml_parser_1.XMLParser({
                             ignoreDeclaration: true,
-                            ignorePiTags: true
+                            ignorePiTags: true,
+                            ignoreAttributes: false
                         });
                         parsedBody = parser.parse(body.toString());
                     }
-                    else if (responseContentType.includes('image/')) {
+                    else if (responseContentType.includes('image')) {
                         parsedBody = body.toString('base64');
                     }
                     if (response.statusCode !== undefined && response.statusCode >= 200 && response.statusCode < 300) {
@@ -144,7 +145,7 @@ function doHttpsRequest(options, resolve, reject) {
 function setRequestBody(request, data) {
     if (data) {
         const requestContentType = request.getHeader('content-type')?.toString() ?? '';
-        if (requestContentType === '' || requestContentType.includes('application/json')) {
+        if (requestContentType === '' || requestContentType.includes('json')) {
             //We serialize using JSON.stringify as a default, or if JSON is specified
             request.write(JSON.stringify(data));
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uoa/lambda-tracing",
-  "version": "2.1.0",
+  "version": "2.1.2",
   "description": "Library for logging & distributed tracing in UoA Lambda projects",
   "repository": {
     "type": "git",

package/uoaHttps.ts

@@ -103,15 +103,16 @@ function doHttpsRequest(options: RequestOptions, resolve: any, reject: any): Cli
 
                     //Get the response content-type header value so we can apply different parsing methods
                     const responseContentType: string = response.headers["content-type"]?.toLowerCase() ?? '';
-                    if (responseContentType === '' || responseContentType.includes('application/json')) {
+                    if (responseContentType === '' || responseContentType.includes('json')) {
                         parsedBody = JSON.parse(body.toString());
-                    } else if (responseContentType.includes('application/xml') || responseContentType.includes('text/xml')) {
+                    } else if (responseContentType.includes('xml')) {
                         const parser = new XMLParser({
                             ignoreDeclaration: true,
-                            ignorePiTags: true
+                            ignorePiTags: true,
+                            ignoreAttributes: false
                         });
                         parsedBody = parser.parse(body.toString());
-                    } else if (responseContentType.includes('image/')) {
+                    } else if (responseContentType.includes('image')) {
                         parsedBody = body.toString('base64');
                     }
 
@@ -135,7 +136,7 @@ function doHttpsRequest(options: RequestOptions, resolve: any, reject: any): Cli
 function setRequestBody(request: ClientRequest, data: any) {
     if (data) {
         const requestContentType: string = request.getHeader('content-type')?.toString() ?? '';
-        if (requestContentType === '' || requestContentType.includes('application/json')) {
+        if (requestContentType === '' || requestContentType.includes('json')) {
             //We serialize using JSON.stringify as a default, or if JSON is specified
             request.write(JSON.stringify(data));
         } else {