awx icon indicating copy to clipboard operation
awx copied to clipboard

Support Django logout redirects

Open manstis opened this issue 10 months ago • 1 comments

SUMMARY

This PR allows specification of Django's auth_views.LogoutView.success_url_allowed_hosts from settings.

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
AWX VERSION
awx: 24.3.1.dev9+g4d163c3330
ADDITIONAL INFORMATION

This has been discussed on #aap-controller on Slack.

I managed to step through the code with sdb to check the settings change has no ill affect.

This PR will need an accompanying PR for awx-controller to be able to set the new environment variable from then operator.

This part of the code-base seems sparse of unit tests.. IDK if you require some?

manstis avatar Apr 26 '24 09:04 manstis

I probably don't even need to change awx-operator as we can set the new environment variable as an extra_setting unless you want to make the new environment variable more public (e.g. include in the CRD/CSV as an explicit parameter)..

manstis avatar Apr 26 '24 18:04 manstis