Removal of Third Party Auth #15571
Merged
Removal of Third Party Auth #15571
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
component:api
component:awx_collection
TheRealHaoLiu
force-pushed
the
feature_3rd-party-auth-removal
branch
3 times, most recently
from
a18388d
to
7a665a2
Compare
hakbailey
suggested changes
Oct 3, 2024
There was a problem hiding this comment.
In addition to the inline comments, we should remove the part of the requirements README that references social-auth-app-django, which has been removed.
hakbailey
approved these changes
Oct 14, 2024
jessicamack
force-pushed
the
feature_3rd-party-auth-removal
branch
from
87e010d
to
9f9fe3e
Compare
|
I see sso app migrations deleted. I can see this working for a new install but does it break upgrading? I looked for, but didn’t find, the deletion of the sso app from the Django app list. On mobile right now, going to look again on the desktop and update my comment. edit: I see that the I approve this PR. |
chrismeyersfsu
approved these changes
Oct 15, 2024
TheRealHaoLiu
force-pushed
the
feature_3rd-party-auth-removal
branch
2 times, most recently
from
4eed8c2
to
5d80cad
Compare
Remove LDAP authentication from AWX
Remove TACACS+ authentication from AWX. Co-authored-by: Hao Liu <[email protected]>
Remove RADIUS authentication from AWX Do not remove models fields and tables let it for a stage where all the work of removing external auth finished AAP-27707 Co-authored-by: Hao Liu <[email protected]>
Remove social oauth (Azure, Github, Google) Co-authored-by: jessicamack <[email protected]>
remove keycloak
* Removed docs associated with SAML auth. * Removed more mentions about SAML. --------- Co-authored-by: jessicamack <[email protected]>
* remove oidc * remove test fields, linting fix * merge commit
* remove saml * remove license file and management command * update requirements, add migrations * remove unused imports
Remove sso app.
- Sequentiallize auth config removal migrations - Remove references to third party auth - update license files - lint fix - Remove unneeded docs - Remove unreferenced file - Remove social auth references from docs - Remove rest of sso dir - Remove references to third part auth in docs - Removed screenshots of UI listing removed settings - Remove AuthView references - Remove unused imports ... Co-Authored-By: jessicamack <[email protected]>
TheRealHaoLiu
force-pushed
the
feature_3rd-party-auth-removal
branch
from
5d80cad
to
d942da1
Compare
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
As referenced here: https://forum.ansible.com/t/transitioning-authentication-and-authorization-rbac-to-the-new-awx-architecture
ISSUE TYPE
COMPONENT NAME
AWX VERSION
ADDITIONAL INFORMATION