Merge pull request #5 from AzureAD/fix-conflictexception

Fixed exception handling for HTTP errors thrown by the provider
AzureAD · Feb 25, 2020 · a074698 · a074698
2 parents 145339d + b993367
commit a074698
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/Microsoft.SystemForCrossDomainIdentityManagement/Service/Controllers/ControllerTemplate.cs b/Microsoft.SystemForCrossDomainIdentityManagement/Service/Controllers/ControllerTemplate.cs
@@ -609,6 +609,23 @@ public virtual async Task<ActionResult<Resource>> Post([FromBody]T resource)
 
                 throw new HttpResponseException(HttpStatusCode.NotImplemented);
             }
+            catch (HttpResponseException httpResponseException)
+            {
+                if (this.TryGetMonitor(out IMonitor monitor))
+                {
+                    IExceptionNotification notification =
+                        ExceptionNotificationFactory.Instance.CreateNotification(
+                            httpResponseException,
+                            correlationIdentifier,
+                            ServiceNotificationIdentifiers.ControllerTemplatePostNotSupportedException);
+                    monitor.Report(notification);
+                }
+
+                if (httpResponseException.Response.StatusCode == HttpStatusCode.Conflict)
+                    return this.Conflict();
+                else
+                    return this.BadRequest();
+            }
             catch (Exception exception)
             {
                 if (this.TryGetMonitor(out IMonitor monitor))