Merge branch 'dev' of https://github.com/dotnet/eShopOnContainers into dev

src/Services/Ordering/Ordering.API/Application/Decorators/ValidatorDecorator.cs

@@ -1,5 +1,6 @@
 using FluentValidation;
 using MediatR;
+using Ordering.Domain.Exceptions;
 using System;
 using System.Collections.Generic;
 using System.Linq;
@@ -33,8 +34,8 @@ namespace Ordering.API.Application.Decorators
 
             if (failures.Any())
             {
-                throw new ValidationException(
-                    $"Command Validation Errors for type {typeof(TRequest).Name}", failures);
+                throw new OrderingDomainException(
+                    $"Command Validation Errors for type {typeof(TRequest).Name}", new ValidationException("Validation exception", failures));
             }
             
             var response = await _inner.Handle(message);

src/Services/Ordering/Ordering.API/Infrastructure/Filters/HttpGlobalExceptionFilter.cs

@@ -32,6 +32,8 @@
                     Messages = new[] { context.Exception.Message }
                 };
 
+                // Result asigned to a result object but in destiny the response is empty. This is a known bug of .net core 1.1
+                //It will be fixed in .net core 1.1.2. See https://github.com/aspnet/Mvc/issues/5594 for more information
                 context.Result = new BadRequestObjectResult(json);
                 context.HttpContext.Response.StatusCode = (int)HttpStatusCode.BadRequest;
             }
@@ -47,6 +49,8 @@
                     json.DeveloperMeesage = context.Exception;
                 }
 
+                // Result asigned to a result object but in destiny the response is empty. This is a known bug of .net core 1.1
+                // It will be fixed in .net core 1.1.2. See https://github.com/aspnet/Mvc/issues/5594 for more information
                 context.Result = new InternalServerErrorObjectResult(json);
                 context.HttpContext.Response.StatusCode = (int)HttpStatusCode.InternalServerError;                
             }