Retire the vast majority of code #218
Conversation
|
My idea is to wait for FAS retirement to be imminent and Bodhi have moved to OIDC before releasing a version with this patch merged. |
|
I'd keep around the OpenID base client as the FAS clients for now. Basically, I'd say baby steps :) It would be easier to do backports/fixes if the set of changes is lower, we can always open a new PR dropping these other files when the time to make that release comes |
|
Hi @puiterwijk I just asked @ralphbean if I should take care of issue 186 |
|
Where are we for this patch? Having all this code dropped would make it much easier to package for Mageia and openSUSE... |
This drops the code for the FAS auth modules for: - TurboGears1 - TurboGears2 - Django All applications should have moved over to OpenID (Connect) by now. This drops the FAS client, FAS is going away for Fedora rather soon. This drops the OpenID Base clients: for API authentication, applications need to switch to OpenID Connect. At ths moment, Bodhi is the one client I am aware of that uses this, and it will be moved to OIDC soon. Signed-off-by: Patrick Uiterwijk <[email protected]>
|
The current Fedora bodhi package uses fedora/client/openidbaseclient.py (at least in tests). |
This drops the code for the FAS auth modules for:
All applications should have moved over to OpenID (Connect) by now.
This drops the FAS client, FAS is going away for Fedora rather soon.
This drops the OpenID Base clients: for API authentication, applications
need to switch to OpenID Connect.
At ths moment, Bodhi is the one client I am aware of that uses this,
and it will be moved to OIDC soon.
Signed-off-by: Patrick Uiterwijk [email protected]