Fix Bug in Redirection Host Schema preventing auto redirection hosts

[cal-tec]

This commit fixes bug's #4692 and #4080 A previous attempt to fix this issue ( commit d96a398 ) referenced the wrong string from the ui.

[nginxproxymanagerci[bot]]

Docker Image for build 1 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-4751

[!NOTE] Ensure you backup your NPM instance before testing this image! Especially if there are database changes. This is a different docker image namespace than the official image.

[!WARNING] Changes and additions to DNS Providers require verification by at least 2 members of the community!

[Spacelord09]

Can anyone merge this?

[ebirbe]

Hello @jc21 are you the right person to review / merge this?

[spotter-mmf]

I'm running into this issue with redirection hosts as well. Hoping this will fix the problem, if we can get the change merged?

[jc21]

Can you test with the new react UI:

nginxproxymanager/nginx-proxy-manager-dev:develop

and let me know if it's still an issue?

[spotter-mmf]

Works without an issue for me in that build!

[Spacelord09]

#4692

[jc21]

I've fixed this differently, by using auto for the UI and db column and rendering $scheme in the nginx config.

The reason for this: I don't like the idea of having this value being set in the nginx config directly from database data without any validation. I also don't like using dollar variables as values.

[jc21]

Thanks for your contribution though! I do appreciate the effort!