Make middleware async capable

urbanplatform / django-keycloak-auth

Middleware to allow authorization using Keycloak and Django for django-rest-framework (DRF). This package should only be used in projects starting from scratch, since it overrides the users' management.

MIT License

32 stars 14 forks source link

Make middleware async capable #23

Closed moritz89 closed 1 year ago

moritz89 commented 2 years ago

Why:

Improve performance
Make entire middleware stack async capable

This change addresses the need by:

Refactoring the KeycloakMiddleware to use the MiddlwareMixin

Closes #22

moritz89 commented 2 years ago

This would be a useful feature, especially when using an ASGI server such as Daphne or uvicorn.

moritz89 commented 1 year ago

@uw-rvitorino @simao-silva Could you have a look at this? This is the last MR that would allow me to remove my fork.

It basically adds support for ASGI servers while retaining support for WSGI servers.

diogosilva30 commented 1 year ago

I'm closing this PR, since it has been integrated in #39. Thanks for this change @moritz89