Resource identifier should be placed as a path parameter

aspnetrun / run-aspnetcore-microservices

Microservices on .NET platforms used ASP.NET Web API, Docker, RabbitMQ, MassTransit, Grpc, Yarp API Gateway, PostgreSQL, Redis, SQLite, SqlServer, Marten, Entity Framework Core, CQRS, MediatR, DDD, Vertical and Clean Architecture implementation with using latest features of .NET 8 and C# 12

https://www.udemy.com/course/microservices-architecture-and-implementation-on-dotnet/?couponCode=SEPT24

MIT License

2.7k stars 1.46k forks source link

Resource identifier should be placed as a path parameter #25

Open wypiorsebastian opened 3 years ago

wypiorsebastian commented 3 years ago

I know it's not a course about RESTful principles but maybe you'll consider in the future extracting resource id from request body to path parameter ?

https://github.com/aspnetrun/run-aspnetcore-microservices/blob/c30712aa099bc335f5613a3b196ea1e943f800a5/src/Services/Catalog/Catalog.API/Controllers/CatalogController.cs#L85

mehmetozkaya commented 3 years ago

yes you exactly right, this is good to fix in this issue. I will check it later thanks.

wypiorsebastian commented 3 years ago

No problem. Glad I could help. BTW - great course :)