MaximRouiller / MaximeRouiller.Azure.AppService.EasyAuth

.NET Core integration of Azure AppService EasyAuth
MIT License
54 stars 9 forks source link

[Authorize(AuthenticationSchemes = "EasyAuth")] - Scheme is not really required #2

Closed ghost closed 4 years ago

ghost commented 4 years ago

Mentioning the scheme name in the attribute is not really required. One can simply decorate the controller (or specific actions within) using just [Authorize] and it works perfectly (when not using your library). So why the need to add the property value?

dasiths commented 4 years ago

This is correct. It used the default scheme when the Authorize Filter is present. Look how I've done it here https://github.com/dasiths/NEasyAuthMiddleware