Improve RestError output in Node (#7454)

Log message and type correctly
AzureSDKAutomation · Feb 24, 2020 · 2e45647 · 2e45647
1 parent 0e8ce69
commit 2e45647
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/sdk/core/core-http/lib/restError.ts b/sdk/core/core-http/lib/restError.ts
@@ -25,6 +25,7 @@ export class RestError extends Error {
     response?: HttpOperationResponse
   ) {
     super(message);
+    this.name = "RestError";
     this.code = code;
     this.statusCode = statusCode;
     this.request = request;
@@ -37,6 +38,6 @@ export class RestError extends Error {
    * Logging method for util.inspect in Node
    */
   [custom](): string {
-    return errorSanitizer.sanitize(this);
+    return `RestError: ${this.message} \n ${errorSanitizer.sanitize(this)}`;
   }
 }
diff --git a/sdk/core/core-http/lib/util/sanitizer.ts b/sdk/core/core-http/lib/util/sanitizer.ts
@@ -72,6 +72,14 @@ export class Sanitizer {
   }
 
   private replacer(key: string, value: unknown) {
+    // Ensure Errors include their interesting non-enumerable members
+    if (value instanceof Error) {
+      return {
+        ...value,
+        name: value.name,
+        message: value.message
+      };
+    }
     if (key === "_headersMap") {
       return this.sanitizeHeaders(key, value as {});
     } else if (key === "url") {