Migrate to modern keycloak

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

Migrate to modern keycloak #78

Closed moritz89 closed 1 year ago

moritz89 commented 1 year ago

Why:

Supporting modern and legacy is a lot of overhead

This change addresses the need by:

Updating to only modern >17 versions

moritz89 commented 1 year ago

@uw-rvitorino I don't know at what state your Keycloak migration is, but this is something that has to be done sooner or later. Tried adding support for legacy and modern Keycloak, but it was too much overhead for me. Also updated the minimum Python version, so that we can support Django >4.x

uw-rvitorino commented 1 year ago

@uw-rvitorino I don't know at what state your Keycloak migration is, but this is something that has to be done sooner or later. Tried adding support for legacy and modern Keycloak, but it was too much overhead for me. Also updated the minimum Python version, so that we can support Django >4.x

Hey @moritz89, we are not using this library anymore (at least my team), we've moved on to using mozilla-django-oidc to use OAuth2 (OIDC) so I'm happy to encourage you to push this project further and feel free to upgrade it to more modern versions of Keycloak.

moritz89 commented 1 year ago

Good to know. I'd fork it and then continue it under a new name. When all's up and running might make sense to add a deprecation notice here and a link to where development will be continuing

github-actions[bot] commented 1 year ago

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] commented 1 year ago

This PR was closed because it has been stalled for 10 days with no activity.