Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable service redirect auth and reverse-sync from DAB #15489

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

AlanCoding
Copy link
Member

@AlanCoding AlanCoding commented Sep 4, 2024

SUMMARY

The resource-server redirect auth class was recently added in DAB.

Also, with some recent changes by me, RESOURCE_SERVER_SYNC_ENABLED will be a no-op without the RESOURCE_SERVER defined, making this safe in all configurations.

The service auth class is described in:

https://github.com/ansible/django-ansible-base/blob/devel/docs/apps/service_backed_sso.md

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API

@h-kataria
Copy link

@elyezer please review

Copy link
Member

@elyezer elyezer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should be good to merge this one, we are not tracking devel for the nightly builds.

@AlanCoding
Copy link
Member Author

I will merge when the other corresponding PRs are set to merge. For now that is only ansible/galaxy_ng#2252

@AlanCoding AlanCoding changed the title Update settings from DAB features Enable service redirect auth from DAB Sep 13, 2024
Copy link

sonarcloud bot commented Sep 16, 2024

@AlanCoding AlanCoding changed the title Enable service redirect auth from DAB Enable service redirect auth and reverse-sync from DAB Sep 16, 2024
@AlanCoding AlanCoding merged commit 5b7a050 into ansible:devel Sep 23, 2024
20 checks passed
djyasin pushed a commit to djyasin/awx that referenced this pull request Nov 11, 2024
* Update settings from DAB features

* Move to the end of the list more correctly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants