nodejsで同時にリクエストを処理し、1つのres.sendだけを使ってレスポンスを渡します。

Question

私は、メディアのIDを含むXMLの入力を受け取り、与えられたIDの詳細をXML形式で出力するAPIを開発しています。 2番目の同時リクエストの応答を送信しているときに問題に直面しています。ここで2番目のリクエストは、郵便配達員の「積載」を示すループに入ります。

私がやっているのは、の関数を呼び出してメディアを解析し、コールバックで出力し、res.sendを使って送信しますが、1回の要求に対してのみ機能します。

私はres.sendを使用しているので、同じAPIへの並列要求をループするか、送信後にヘッダーを設定できませんが、res.sendが応答を送信するために使用できる唯一の方法です次は動作しません）。

var getCompositeData = function(req, res, next){ 
    abc.getData(req.body, function(err, xmlOutput){ 
    if(err){ 
    console.log("error"); 
    } else { 
    xmlData = xmlOutput 
    return next() 
    } 
} 

app.post(apiUrl, [ 
    rawBodyParser({ 
    type: 'application/xml' 
    }), 
    app.oauth.authorise() 
], getCompositeData, function (req, res) { 
    res.setHeader('Content-Type', 'application/xml'); 
    res.send(xmlData); 
}); 

Answer 1

あなたのコードにはいくつかの問題があります：エラーが発生した場合

if (err) { 
    console.log("error"); 
} 

が、あなたはまだ、タイムアウトが発生するまでそれ以外の要求は停止します、応答が送り返されることを確認する必要があります。あなたはnextにエラーを渡すことができ、およびExpressは、それを処理します：

if (err) { 
    return next(err); 
} 

次の問題：

xmlData = xmlOutput 

xmlDataが要求ごとに上書きされ宣言されていない変数、です。 （ほとんどの場合）2つのリクエストが（ほとんど）同じ時刻に発生した場合、1つのクライアントが不正な応答を返す可能性があります（Node.jsはJSコードを1つのスレッドで実行し、スレッドローカルストレージはないので、xmlDataはすべてのリクエスト）。

「ストア」には良い場所この種のデータはres.localsである：

res.locals.xmlData = xmlOutput; 
return next(); 

// and later: 
res.send(res.locals.xmlData);