Explorar o código

Use 301 status for redirects

poeti8 %!s(int64=7) %!d(string=hai) anos
pai
achega
d5e31324e9
Modificáronse 2 ficheiros con 3 adicións e 3 borrados
  1. 2 2
      server/controllers/urlController.js
  2. 1 1
      server/server.js

+ 2 - 2
server/controllers/urlController.js

@@ -142,7 +142,7 @@ exports.goToUrl = async (req, res, next) => {
   redis.set(id + (domain || ''), JSON.stringify(url), 'EX', 60 * 60 * 1);
 
   if (url.banned) {
-    return res.redirect('/banned');
+    return res.redirect(301, '/banned');
   }
 
   const doesRequestInfo = /.*\+$/gi.test(reqestedId);
@@ -184,7 +184,7 @@ exports.goToUrl = async (req, res, next) => {
       referrer: referrer || 'Direct',
     });
   }
-  return res.redirect(url.target);
+  return res.redirect(301, url.target);
 };
 
 exports.getUrls = async ({ query, user }, res) => {

+ 1 - 1
server/server.js

@@ -65,7 +65,7 @@ app.prepare().then(() => {
       headers.host !== config.DEFAULT_DOMAIN &&
       (path === '/' || preservedUrls.some(item => item === path.replace('/', '')))
     ) {
-      return res.redirect(`http://${config.DEFAULT_DOMAIN + path}`);
+      return res.redirect(301, `http://${config.DEFAULT_DOMAIN + path}`);
     }
     return next();
   });