Copilot commented on code in PR #2015: URL: https://github.com/apache/apisix-website/pull/2015#discussion_r3058286135
########## .htaccess: ########## @@ -41,10 +41,10 @@ Redirect 302 "/slack" "https://join.slack.com/t/the-asf/shared_invite/zt-1ugrg37 Redirect 302 "/community-meeting-signup" "https://docs.google.com/forms/d/1C9bIJ3eh0bQrBdv4rPGxHDUvX4giNQ_IRCmDDOQ2mgE/"; Redirect 302 "/contributor-workshop-signup" "https://docs.google.com/forms/d/1LUER3R9-aFsUm7MhjVd_CM1xAGnkuWIe62prFH5aqAE/"; Redirect 302 "/guest-blog-post" "https://forms.gle/unQpSm7FyqkfaSSP8/"; -RedirectMatch 302 "^/docs/apisix/getting-started/$" "/docs/apisix/getting-started/README/" -RedirectMatch 302 "^/zh/docs/apisix/getting-started/$" "/zh/docs/apisix/getting-started/README/" -RedirectMatch 302 "^/docs/apisix/3\.2/plugins/(.*)$" "https://apache-apisix.netlify.app/docs/apisix/3.2/plugins/$1"; -RedirectMatch 302 "^/zh/docs/apisix/3\.2/plugins/(.*)$" "https://apache-apisix.netlify.app/zh/docs/apisix/3.2/plugins/$1"; +RedirectMatch 301 "^/docs/apisix/getting-started/$" "/docs/apisix/getting-started/README/" +RedirectMatch 301 "^/zh/docs/apisix/getting-started/$" "/zh/docs/apisix/getting-started/README/" +RedirectMatch 301 "^/docs/apisix/3\.2/plugins/(.*)$" "https://apache-apisix.netlify.app/docs/apisix/3.2/plugins/$1"; +RedirectMatch 301 "^/zh/docs/apisix/3\.2/plugins/(.*)$" "https://apache-apisix.netlify.app/zh/docs/apisix/3.2/plugins/$1"; Review Comment: Switching these redirects from 302 to 301 makes them permanently cacheable by browsers and search engines. If the Netlify target host/path is not guaranteed to be long-lived (or could change again), consider keeping a temporary redirect (302/307) to avoid hard-to-revert caching behavior; otherwise, document why a permanent redirect is intended here. ```suggestion RedirectMatch 302 "^/docs/apisix/3\.2/plugins/(.*)$" "https://apache-apisix.netlify.app/docs/apisix/3.2/plugins/$1"; RedirectMatch 302 "^/zh/docs/apisix/3\.2/plugins/(.*)$" "https://apache-apisix.netlify.app/zh/docs/apisix/3.2/plugins/$1"; ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
