From 09d3724b31229c89e1025e82f301ff5eacfad7b2 Mon Sep 17 00:00:00 2001
From: Bianco Veigel <bianco.veigel@ahd.de>
Date: Fri, 19 May 2017 18:55:20 +0200
Subject: [PATCH] add missing summary comments

---
 Graphite/Exceptions/HttpRequestException.cs   | 33 +++++++++++++++++++
 .../HttpResponseMessageExtension.cs           |  6 ++++
 2 files changed, 39 insertions(+)

diff --git a/Graphite/Exceptions/HttpRequestException.cs b/Graphite/Exceptions/HttpRequestException.cs
index 6212ddd..a32b6a0 100644
--- a/Graphite/Exceptions/HttpRequestException.cs
+++ b/Graphite/Exceptions/HttpRequestException.cs
@@ -1,32 +1,61 @@
 using System;
+using System.Diagnostics.CodeAnalysis;
 using System.Net;
 using System.Runtime.Serialization;
 
 namespace ahd.Graphite.Exceptions
 {
+    /// <summary>
+    /// A base class for exceptions thrown by the <see cref="System.Net.Http.HttpClient"/> and <see cref="System.Net.Http.HttpMessageHandler"/> classes.
+    /// </summary>
     [Serializable]
     public class HttpRequestException: System.Net.Http.HttpRequestException
     {
+        /// <summary>
+        /// Initializes a new instance of the HttpRequestException class.
+        /// </summary>
+        /// <param name="statuscode">Statuscode of the response</param>
+        /// <param name="response">response content</param>
         public HttpRequestException(HttpStatusCode statuscode, string response):base()
         {
             StatusCode = statuscode;
             Response = response;
         }
 
+        /// <summary>
+        /// Initializes a new instance of the HttpRequestException class with a specific message that describes the current exception
+        /// </summary>
+        /// <param name="response">response content</param>
+        /// <param name="message">exception message</param>
         public HttpRequestException(string response, string message):base(message)
         {
             Response = response;
         }
 
+        /// <summary>
+        /// Initializes a new instance of the HttpRequestException class with a specific message that describes the current exception and an inner exception.
+        /// </summary>
+        /// <param name="response">response content</param>
+        /// <param name="message">exception message</param>
+        /// <param name="inner">inner exception</param>
         public HttpRequestException(string response, string message, Exception inner):base(message, inner)
         {
             Response = response;
         }
 
+        /// <summary>
+        /// statuscode of the HTTP response
+        /// </summary>
         public HttpStatusCode StatusCode { get; private set; }
 
+        /// <summary>
+        /// Content of the HTTP response
+        /// </summary>
         public string Response { get; private set; }
 
+        /// <summary>
+        /// Security Critical. (Inherited from Exception.)
+        /// </summary>
         public override void GetObjectData(SerializationInfo info, StreamingContext context)
         {
             if (info == null)
@@ -40,6 +69,10 @@ public override void GetObjectData(SerializationInfo info, StreamingContext cont
             base.GetObjectData(info, context);
         }
 
+        /// <summary>
+        /// Initializes a new instance of the HttpRequestException class.
+        /// </summary>
+        [SuppressMessage("Microsoft.Usage", "CA2236:CallBaseClassMethodsOnISerializableTypes", Justification = "base ctor is missing - blame Microsoft")]
         protected HttpRequestException(SerializationInfo info, StreamingContext context)
         {
             Response = info.GetString(nameof(Response));
diff --git a/Graphite/Exceptions/HttpResponseMessageExtension.cs b/Graphite/Exceptions/HttpResponseMessageExtension.cs
index 867fae9..97713df 100644
--- a/Graphite/Exceptions/HttpResponseMessageExtension.cs
+++ b/Graphite/Exceptions/HttpResponseMessageExtension.cs
@@ -3,8 +3,14 @@
 
 namespace ahd.Graphite.Exceptions
 {
+    /// <summary>
+    /// Extensions for <see cref="HttpResponseMessage"/>
+    /// </summary>
     public static class HttpResponseMessageExtension
     {
+        /// <summary>
+        /// checks the statuscode and throws a <see cref="HttpRequestException"/> with the full response content
+        /// </summary>
         public static async Task EnsureSuccessStatusCodeAsync(this HttpResponseMessage response)
         {
             if (response.IsSuccessStatusCode)