[ziolo86]

#nodejs #javascript

Chciałem zapisać w logu połączenie ze stroną więc napisałem middleware expressa, która zostaje uruchomiona dla adresu /:
app.use('[/]', logger);

w loggerze pobieram aktualną godzinę new Date(), ale niestety jest ona stała w każdym wpisie w logu po restarcie. Takiego problemu nie mam gdy robię zapis do loga w metodzie get routera.

[o.....l]

@ziolo86: Pokaż kod ( ͡° ͜ʖ ͡°)

[ziolo86]

@obereczekpl: fs.appendFile('./logs/access.log', new Date()+ ' ' +clientIP + "\r\n");

[o.....l]

new Date()+ ' ' +clientIP

@ziolo86: To teraz kwestia tego, gdzie uruchamiasz ten fragment. Jeśli wykonujesz jakąś metodę, to new Date() powinni się wykonać za każdym razem, gdy skorzystasz z metody.

Według Twojego opisu wygląda to tak jakbyś podczas ładowania modułu przygotował callback z zapisem do pliku i datą a potem Node.js #!$%@? odwołanie do modułu i zostaje ta sama data.

[ziolo86]

@obereczekpl: tak wygląda moduł logger

module.exports = function(request, response, next) {
 var fs = require('fs');
 var clientIP = (request.header('x-forwarded-for') || request.connection.remoteAddress).substring(7);

 response.on('finish', function () {
 fs.appendFile('./logs/access.log', Date.now()+ ' ' +clientIP + "\r\n", function(err) { console.log("Error zapisu");});
 //console.log('adres: ' + request.url);
 });

 next();
};

[ziolo86]

@obereczekpl: ok czyli wg dokumentacji:

If you want to have a module execute code multiple times, then export a function, and call that function.

[larvaexotech]

@ziolo86: Nie prościej użyć gotowego modułu? https://www.npmjs.com/package/accesslog

var AccessLog = require( 'accesslog' );
var app = Express();
app.use( AccessLog( {
 path: './logs/access.log' ,
} ) );

[o.....l]

Date.now()

@ziolo86: Date.now() = new Date.now()

[o.....l]

@larvaexotech: Pewnie można, ale wydaje mi się, że kolega się uczy i woli sam zrobić ( ͡° ͜ʖ ͡°)

[ziolo86]

@obereczekpl: @larvaexotech:
dzięki za pomoc ( ͡° ͜ʖ ͡°)

[o.....l]

@ziolo86: tam to now() odpuść.

Jeśli chodzi o stringa z datą to zapisz to tak:

(new Date()).toString();

A jeśli o timestamp to tak:

(+new Date())

[ziolo86]

@obereczekpl: tak usunąłem :)