Избегайте пролета вокруг объекта запроса

Question

Я пишу приложение React (Экспресс-сервер), который отображает на сервере клиента + с использованием общей ('изоморфной') кодовой базы. В некоторых случаях, когда сервер является средством визуализации, я хочу получить доступ к объекту запроса, не передавая его по всему месту. Поскольку узел отвечает на один запрос за раз, небезопасно ли использовать кеширование переменной, не забудьте сбросить ее перед отправкой ответа?

Это работает, но может кто-то сказать мне, почему это либо хорошо или плохая идея? ...

server.js

var r = require('./request-store'); 
app.use(function(req, res, next) { 
    r.set(req); 
    next(); 
}); 

var resolve = require('./resolve'); 
// ...deferring to react router for client/server routing 
app.use(function(req, res) { 
    resolve().then(function(data) { 
    r.set(null); 
    res.render('index', { data: data }); 
    }); 
}); 

запроса store.js

var r = null; 
module.exports.set = function(req) { r = req; }; 
module.exports.get = function() { return r; }; 

resolve.js

var r = require('./request-store'); 
module.exports = function() { 
    return new Promise(resolve, reject) { 
    if (isServer) { 
     console.log(r.get()); 
     resolve(); 
    } else if (isClient) { 
     resolve(); 
    } 
    } 
}; 

Answer 1

Но что произойдет, если вы получите два одновременных запроса от разных пользователей? Первый запрос будет кэшироваться, а второй будет перезаписывать запрос первого пользователя. Между тем предположим, что вы выполняли несколько операций ввода-вывода и нуждались в доступе к запросу после его завершения, тогда объект запроса первого пользователя уже был бы перезаписан вторым запросом пользователя. В целом, это очень опасно. Лучшим способом было бы использовать какое-то хранилище в хранилище ключей памяти, но вам все равно нужно каким-то образом обойти ключ.

Answer 2

Дано Node однопоточное, и вы не : для выполнения любых операций ввода-вывода, то да, ваш код выглядит безопасным для меня. Express будет запускать все промежуточное программное обеспечение синхронно, поэтому к моменту вашего обещания (как и синхронно) r.get() должен вернуть правильный объект запроса.

Однако, я не уверен, что вам нужно кэшировать на всех, вы можете передать его непосредственно в resolve и использовать замыкание, чтобы захватить его в свое обещание

var resolve = require('./resolve'); 
// ...deferring to react router for client/server routing 
app.use(function(req, res) { 
    // pass 'req' into `resolve` 
    resolve(req).then(function(data) { 
     r.set(null); 
     res.render('index', { data: data }); 
    }); 
}); 

resolve.js

var r = require('./request-store'); 
module.exports = function(req) { 
    // capture 'req' for use later 
    return new Promise(resolve, reject) { 
    if (isServer) { 
     console.log(req); 
     resolve(); 
    } else if (isClient) { 
     resolve(); 
    } 
    } 
};