permanent redirect status code instead of found

[francesco-gaglione]

PR Type

Close #3543

PR Checklist

• Tests for the changes have been added / updated.
• Documentation comments have been added / updated.
• A changelog entry has been made for the appropriate packages.
• Format code with the latest stable rustfmt.
• (Team) Label with affected crates and semver status.

Overview

[robjtede]

I agree Found is less appropriate but permanent redirects are a risky default for a framework due to their caching implications.

I'd accept a change to TemporaryRedirect or allow a customization.

[francesco-gaglione]

@robjtede if customization is prefered I can try to work on it. I wa thinking to change redirect_to_slash bolean to an Option containint the redirect mode. What do you think?
I can work on it on a different PR if prefered leaving here the TemporaryRedirect

[robjtede]

change redirect_to_slash

a new method would be preferred to avoid breaking changes, but in spirit i agree it's a good path to take

[francesco-gaglione]

@robjtede working on it. This merge request will be obsolete and will be included in the new one.

[francesco-gaglione]

Opened #3767