Merge pull request #164 from Medium/jamie-permanent-redirect-support

Support permanent redirects

package.json

@@ -1,7 +1,7 @@
 {
   "name": "matador",
   "description": "an MVC framework for Node",
-  "version": "2.0.0-alpha.9",
+  "version": "2.0.0-alpha.10",
   "homepage": "https://github.com/Medium/matador",
   "main": "src/matador.js",
   "authors": [

src/matador.js

@@ -415,6 +415,12 @@ function requestDecorator(app, err, req, res, next) {
     res.end()
   }
 
+  // Add a method to permanently redirect the response to a new url.
+  res.redirectPermanent = function redirectRequestPermanent(url) {
+    res.writeHead(301, {'Location': url})
+    res.end()
+  }
+
   // Add a method for sending output to the response. Defaults to HTML.
   res.send = function sendResponse(data, headers, status) {
     var bytesWritten

tests/integration/request_response_decorators/app/config/routes.js

@@ -1,6 +1,7 @@
 module.exports = function (app) {
   return {
     '/redirect': 'Home.redirect',
+    '/redirectPermanent': 'Home.redirectPermanent',
     '/target': 'Home.target',
     '/queryStrings': 'Home.queryStrings'
   }

tests/integration/request_response_decorators/app/controllers/HomeController.js

@@ -10,6 +10,10 @@ module.exports = function (app, config) {
 
   util.inherits(HomeController, ApplicationController)
 
+  HomeController.prototype.redirectPermanent = function (req, res) {
+    res.redirectPermanent('/elsewhere')
+  }
+
   HomeController.prototype.redirect = function (req, res) {
     res.redirect('/target')
   }

tests/integration/request_response_decorators_test.js

@@ -2,6 +2,10 @@ var falkor = require('falkor')
 
 var port = process.env.PORT || 3000
 
+exports.testResponsePermanentRedirect = falkor.fetch('http://localhost:' + port + '/redirectPermanent')
+  .expectStatusCode(301)
+  .expectHeader('Location', '/elsewhere')
+
 exports.testResponseRedirect = falkor.fetch('http://localhost:' + port + '/redirect')
   .expectStatusCode(302)
   .expectHeader('Location', '/target')