refactor, add README and fix double hook

Author: fergalmcl

resources/sdk/pureclouddotnet/extensions/Client/AbstractHttpClient.cs

@@ -94,7 +94,7 @@ public void SetPostRequestHook(PostRequestHook hook)
         }
 
         /// <summary>
-        /// Applies the pre-request hook if available
+        /// Applies the pre-request hook if set
         /// </summary>
         /// <param name="request">The original request</param>
         /// <returns>The modified request</returns>
@@ -116,7 +116,7 @@ protected IHttpRequest ApplyPreRequestHook(IHttpRequest request)
         }
 
         /// <summary>
-        /// Applies the post-request hook if available
+        /// Applies the post-request hook if set
         /// </summary>
         /// <param name="response">The original response</param>
         /// <returns>The modified response</returns>

resources/sdk/pureclouddotnet/extensions/Client/DefaultHttpClient.cs

@@ -172,7 +172,6 @@ private bool ValidateServerCertificate(object sender, X509Certificate certificat
         /// </summary>
         public override IHttpResponse Execute(IHttpRequest httpRequest)
         {
-            httpRequest = ApplyPreRequestHook(httpRequest);
             IHttpResponse resp;
 
             if (usingMTLS) //using HttpWebRequest
@@ -181,14 +180,16 @@ public override IHttpResponse Execute(IHttpRequest httpRequest)
             }
             else //using RestSharp (HttpClient)
             {
+                httpRequest = ApplyPreRequestHook(httpRequest);
                 var request = PrepareRestRequest((HttpRequestOptions)httpRequest);
 
                 var restResp = restClient.Execute(request);
 
                 resp = ConvertToHttpResponse(restResp);
+                resp = ApplyPostRequestHook(resp);
             }
 
-            return ApplyPostRequestHook(resp);
+            return resp;
         }
 
         private IHttpResponse ConvertToHttpResponse(RestResponse response)

resources/sdk/pureclouddotnet/templates/README.mustache

@@ -633,6 +633,39 @@ var certPass = "x509Password";
 Configuration.Default.ApiClient.SetMTLSCertificates(certPath, certPass);
 ```
 
+### Using Pre Commit and Post Commit Hooks
+
+For any custom requirements like pre validations or post cleanups (for ex: OCSP and CRL validation), we can inject the prehook and posthook functions.
+The SDK's default client will make sure the injected hook functions are executed.
+
+The Pre/Post Hook functions must have the following method signature shown below
+
+```csharp
+IHttpRequest preHook(IHttpRequest request)
+IHttpResponse postHook(IHttpRepsonse response)
+```
+
+Here is an example of using a Pre Hook function:
+
+```csharp
+private IHttpRequest preHook(IHttpRequest request)
+{
+    try {
+        Console.WriteLine("Running PreHook: Certificate Validation Checks");
+
+        // custom validation here
+
+        Console.WriteLine("Certificate Validation Complete");
+    } catch (Exception ex) {
+        Console.WriteLine($"Error in prehook validation: {ex.Message}");
+        throw ex; // Reject request if validation fails
+    }
+}
+
+Configuration.Default.ApiClient.HttpClient.SetPreRequestHook(preHook);
+
+```
+
 ### Building from Source
 
 If you're working inside Visual Studio, adding the files to your project allows you to edit and build inside an IDE.

resources/sdk/pureclouddotnet/templates/test-SdkTests.mustache

@@ -341,7 +341,7 @@ namespace {{packageName}}.Tests
             Console.WriteLine("Running Pre-Request Hook");
             var options = (HttpRequestOptions)request;
             
-            options.AddHeaderParam("X-Test-Header", "Pre-Hook-Test");
+            options.AddHeaderParam("Test-Header", "Pre-Hook-Test");
             
             Console.WriteLine($"Pre-Hook: Making {options.Method} request to {options.Url}");
             
@@ -509,6 +509,9 @@ namespace {{packageName}}.Tests
             Console.WriteLine($"Deleted user with ID {userId}");
         }
 
+        ///<Summary>
+        /// CleanUp
+        ///</Summary>
         [Test, Order(7)]
         public void CleanUp()
         {