Make service errors more discoverable.

Author: peombwa

src/readme.graph.md

@@ -246,6 +246,27 @@ directive:
                 }
             }
           }
+# Mark '@odata.id' as required properties for /$ref.
+  - from: 'openapi-document'
+    where: $.components.schemas['microsoft.graph.ODataErrors.InnerError']
+    transform: >-
+      return {
+        "type": "object",
+        "properties": {
+            "date" : {
+                "type" : 'string'
+            },
+            "request-id" : {
+                "type" : 'string'
+            },
+            "client-request-id" : {
+                "type" : 'string'
+            }
+        },
+        "additionalProperties": {
+            "type" : "object"
+        }
+      }
 # Mark '@odata.id' as required properties for /$ref.
   - from: 'openapi-document'
     where: $.components.schemas.ReferenceCreate
@@ -394,6 +415,10 @@ directive:
           }
         }
 
+        // Format error details.
+        let errorDetailsRegex = /(ErrorDetails\s*=\s*)(new.*ErrorDetails\(message\).*)/gmi
+        $ = $.replace(errorDetailsRegex, '$1this.GetErrorDetails((await response)?.Error, responseMessage)');
+
         return $;
       }
 
@@ -542,6 +567,21 @@ directive:
         return $;
       }
 
+# Modify generated JsonObject class.
+  - from: source-file-csharp
+    where: $
+    transform: >
+      if (!$documentPath.match(/generated%2Fruntime%2FNodes%2FJsonObject.cs/gm))
+      {
+        return $;
+      } else {
+        // Make JsonObject's items dictionary case insensitive.
+        let dictionaryInitRegex = /(\s*=\s*new\s*Dictionary<string,\s*JsonNode>\()(\);)/gm
+        $ = $.replace(dictionaryInitRegex, '$1StringComparer.InvariantCultureIgnoreCase$2');
+
+        return $;
+      }
+
 # Modify API class.
   - from: source-file-csharp
     where: $

tools/Custom/HttpMessageLogFormatter.cs

@@ -63,9 +63,9 @@ public static async Task<string> GetHttpRequestLogAsync(HttpRequestMessage reque
 
             StringBuilder stringBuilder = new StringBuilder();
             stringBuilder.AppendLine($"============================ HTTP REQUEST ============================{Environment.NewLine}");
-            stringBuilder.AppendLine($"HTTP Method:{Environment.NewLine}{requestClone.Method.ToString()}{Environment.NewLine}");
-            stringBuilder.AppendLine($"Absolute Uri:{Environment.NewLine}{requestClone.RequestUri.ToString()}{Environment.NewLine}");
-            stringBuilder.AppendLine($"Headers:{Environment.NewLine}{HeadersToString(ConvertHttpHeadersToCollection(requestClone.Headers))}{Environment.NewLine}");
+            stringBuilder.AppendLine($"HTTP Method:{Environment.NewLine}{requestClone.Method}{Environment.NewLine}");
+            stringBuilder.AppendLine($"Absolute Uri:{Environment.NewLine}{requestClone.RequestUri}{Environment.NewLine}");
+            stringBuilder.AppendLine($"Headers:{Environment.NewLine}{HeadersToString(requestClone.Headers)}{Environment.NewLine}");
             stringBuilder.AppendLine($"Body:{Environment.NewLine}{SanitizeBody(body)}{Environment.NewLine}");
             return stringBuilder.ToString();
         }
@@ -84,12 +84,17 @@ public static async Task<string> GetHttpResponseLogAsync(HttpResponseMessage res
             StringBuilder stringBuilder = new StringBuilder();
             stringBuilder.AppendLine($"============================ HTTP RESPONSE ============================{Environment.NewLine}");
             stringBuilder.AppendLine($"Status Code:{Environment.NewLine}{response.StatusCode}{Environment.NewLine}");
-            stringBuilder.AppendLine($"Headers:{Environment.NewLine}{HeadersToString(ConvertHttpHeadersToCollection(response.Headers))}{Environment.NewLine}");
+            stringBuilder.AppendLine($"Headers:{Environment.NewLine}{HeadersToString(response.Headers)}{Environment.NewLine}");
             stringBuilder.AppendLine($"Body:{Environment.NewLine}{SanitizeBody(body)}{Environment.NewLine}");
             return stringBuilder.ToString();
         }
 
-        private static Regex regexPattern = new Regex("(\\s*\"access_token\"\\s*:\\s*)\"[^\"]+\"", RegexOptions.Compiled);
+        internal static string HeadersToString(HttpHeaders headers)
+        {
+            return HeadersToString(ConvertHttpHeadersToCollection(headers));
+        }
+
+        private static readonly Regex regexPattern = new Regex("(\\s*\"access_token\"\\s*:\\s*)\"[^\"]+\"", RegexOptions.Compiled);
         private static object SanitizeBody(string body)
         {
             IList<Regex> regexList = new List<Regex>();
@@ -110,7 +115,7 @@ private static IDictionary<string, IEnumerable<string>> ConvertHttpHeadersToColl
             return headers.ToDictionary(a => a.Key, a => a.Value);
         }
 
-        private static object HeadersToString(IDictionary<string, IEnumerable<string>> headers)
+        private static string HeadersToString(IDictionary<string, IEnumerable<string>> headers)
         {
             StringBuilder stringBuilder = headers.Aggregate(new StringBuilder(),
                 (sb, kvp) => sb.AppendLine(string.Format("{0,-30}: {1}", kvp.Key, String.Join(",", kvp.Value.ToArray()))));

tools/Custom/PSCmdletExtensions.cs

@@ -5,6 +5,7 @@ namespace NamespacePrefixPlaceholder.PowerShell
 {
     using Microsoft.Graph.PowerShell.Authentication;
     using Microsoft.Graph.PowerShell.Authentication.Common;
+    using NamespacePrefixPlaceholder.PowerShell.Models;
     using System;
     using System.Collections.ObjectModel;
     using System.IO;
@@ -105,6 +106,21 @@ internal static void WriteToFile(this PSCmdlet cmdlet, HttpResponseMessage respo
             }
         }
 
+        internal static ErrorDetails GetErrorDetails(this PSCmdlet cmdlet, IMicrosoftGraphODataErrorsMainError odataError, HttpResponseMessage responseMessage)
+        {
+            var serviceErrorDoc = "https://learn.microsoft.com/graph/errors";
+            var recommendedAction = $"See service error codes: {serviceErrorDoc}";
+            var requestUri = $"{responseMessage.RequestMessage?.Method} {responseMessage.RequestMessage?.RequestUri}";
+            string date = odataError?.Innererror?.Date;
+            string requestId = odataError?.Innererror?.RequestId;
+            string clientRequestId = odataError?.Innererror?.ClientRequestId;
+            string errorDetailsMessage = $"{odataError?.Message}\n\n{requestUri}\nStatusCode: {responseMessage.StatusCode}\nErrorCode: {odataError?.Code}\nDate: {date}\nRequestId: {requestId}\nClientRequestId: {clientRequestId}\n\n{recommendedAction}";
+            return new ErrorDetails(errorDetailsMessage)
+            {
+                RecommendedAction = recommendedAction
+            };
+        }
+
         /// <summary>
         /// Writes an input stream to an output stream.
         /// </summary>