BUGFIX: use status code 301 in handleShortcutNode when redirecting

[andrehoffmann30]

With this bugfix the status code 301 is used when redirection from a shortcut document node instead of the default 303.

[andrehoffmann30]

Hello dear Neos Community,
here is an explanation of why this change makes sense. The status code 301 is preferable for redirects from an SEO point of view, as it passes on the weighting of the original url to the new url for search engines (keyword linkjuice). The status code 303 should not be used for SEO-relevant redirects. However, many Neos users (such as one of our customers) use the shortcut node type primarily for redirects within a Neos website or for redirects to external but related websites and would like to retain SEO relevance for the redirect url. See also the following articles:

https://seoforum.com/thread/301-redirect-vs-303-redirect

https://webmasters.stackexchange.com/questions/123194/difference-between-301-and-303-status-codes#:~:text=303%20was%20intended%20to%20be%20used%20when%20a%20user%20sends%20a%20GET%20request%20to%20a%20POST%20url.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Status#redirection_messages

I hope the explanations help you. Please contact me if you have any further questions. Thank you very much.

[dlubitz]

Thank you @andrehoffmann30 for the PR. This was already discussed a lot (#4410) and we implemented a configurable solution for Neos 9: #4490

I wouldn't consider this as a bugfix. We could consider backporting this change to 8.4.

[kdambekalns]

I couldn't are less for "link juice", and wil not approve this. Also, see https://web.archive.org/web/20200209130229/https://getluky.net/2010/12/14/301-redirects-cannot-be-undon/ (admittedly someone should verify if that is still valid…)

👍 for bringing #4490 into 8.4, though!