[z.....6]

#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 ( ͡° ͜ʖ ͡°)

[z.....6]

@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 cachuje odwołanie do modułu i zostaje ta sama data.

[z.....6]

@obereczekpl: tak wygląda moduł logger

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

[z.....6]

@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ć ( ͡° ͜ʖ ͡°)

[z.....6]

@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

[z.....6]

@obereczekpl: tak usunąłem :)