Massimo Crippa

Add guidelines about content negotiation

4

provide guidelines on content negotiation (accept / content-type).

Produces attribute

4

You could include the type that is returned by this operation as well (typeof(CarDto[])). Actually, my take would be to specify a Produces attribute that covers the response of a...

Health check middleware

5

consider to use the health check middleware (ASPNET core 2.2) https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/health-checks?view=aspnetcore-2.2

Provide guidance on security

5

Secure the API access + Apikey + Ip Filtering + Certificate + OAuth token (from IDP) + MSI (managed identity) + ...

Serialization and Accept header

2

Shouldn't we let ASP.NET handle the serialization of the error in the format that is specified by the accept header of the request instead of deciding for ourselves that it...

One repository per aggregate root, and not one global repository.

1

You should have a repository per aggregate root, and not one global repository. I think in this domain model, we currently have only one aggregate root, which would be `Car`...