Handle unauthorized exceptions in a custom handler

[lubosmj]

closes #1254

[newswangerd]

Won't this set the exception handler for all plugins?

[lubosmj]

Hmm, that is a good question. I think it probably will. 😭

[lubosmj]

I can try cloning this to make it available only to pulp_container.

[rochacbruno]

Suggested change

	from django.conf import settings
	DRF_ACCESS_POLICY = {
	"dynaconf_merge_unique": True,
	"reusable_conditions": ["pulp_container.app.global_access_conditions"],
	}
	REST_FRAMEWORK = deepcopy(settings.REST_FRAMEWORK)
	REST_FRAMEWORK.update(
	{"EXCEPTION_HANDLER": "pulp_container.app.exceptions.unauthorized_exception_handler"}
	DRF_ACCESS_POLICY__reusable_conditions = "@merge_unique pulp_container.app.global_access_conditions"
	REST_FRAMEWORK__EXCEPTION_HANDLER = "pulp_container.app.exceptions.unauthorized_exception_handler"

[rochacbruno]

I don' t recommend importing django.conf.settings here, you can use dynaconf inline merging with @ and __ (as in my other comment)

DRF_ACCESS_POLICY__reusable_conditions = "@merge_unique pulp_container.app.global_access_conditions"
REST_FRAMEWORK__EXCEPTION_HANDLER = "pulp_container.app.exceptions.unauthorized_exception_handler"

Or you can add a pulp_container/app/dynaconf_hooks.py and then do any conditionals or updates there.

dynaconf_hooks.py

def post(settings):
    data = {"dynaconf_merge": True}
    data["REST_FRAMEWORK"]["EXCEPTION_HANDLER"] = "..."
    return data

The reason is that as settings is formed by the merging of multiple sources the final state is not the one you get grom django.conf at this point here.

[lubosmj]

I am closing this PR in favour of #1256