Location header

[anythingagency]

There is currently no way to pass in a Location: header in the beforeRender. This therefore breaks a cached 301 response, as the status is set to 301, but the Location isn't set.

Need to be able to:

done(err, {body: fields[0], status: fields[1], location: fields[2]});

[thoop]

The beforeRender function starts as follows:

prerender.beforeRenderFn(req, function(err, cachedRender) {

    if (!err && cachedRender) {
      if (typeof cachedRender == 'string') {
        res.writeHead(200, {
          "Content-Type": "text/html"
        });
        return res.end(cachedRender);
      } else if (typeof cachedRender == 'object') {
        res.writeHead(cachedRender.status || 200, {
          "Content-Type": "text/html"
        });
        return res.end(cachedRender.body || '');
      }
    }
...

I think we could probably break this out:

res.writeHead(cachedRender.status || 200, {
    "Content-Type": "text/html"
});

into something like:

var status = cachedRender.status || 200;
var headers = cachedRender.headers || { "Content-Type": "text/html" };
res.writeHead(status, headers);

That would let you override the headers being sent back if you are caching those too. What do you think?

[anythingagency]

Thats would be a perfect solution. Thank you for replying.