openwisp/openwisp-radius

[bug] Site key of organization's password reset url is not formatted automatically

pandafy opened this issue · 1 comments

The default value for OPENWISP_RADIUS_PASSWORD_RESET_URLS contains {site} placeholder. The documentation hints that the value for {site} is taken from the Django site object.

{site}: site domain as defined in the django site framework (defaults to example.com and an be changed through the django admin)

I couldn't find the source code that deals with this. Also, when I try to save a new organization I get the following error

Screenshot from 2022-06-28 16-14-42

The URL field validation fails because of {site} placeholder.

I wonder if we can resolve the {site} and {organization} placeholders only for validation purposes?

Or we could copy the original value, update the value to make the validation pass and then restore the original value before saving.

The first option would be cleaner if doable.