winstonjs/winston

3.0.0 winston.query throws exception for File Transport

Open

#1 130 ouverte le 6 nov. 2017

Voir sur GitHub
 (15 commentaires) (0 réactions) (0 assignés)JavaScript (21 967 stars) (1 789 forks)batch import
BugGood First IssueHelp Wantedwinston-file

Description

The Functions Transport.prototype.normalizeQuery, Transport.prototype.formatResults and common.clone were removed in 3.0.0, so query will throw an unhandled exception.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
backend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic understanding of Node.jsFamiliarity with winston transport API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
The issue reports that methods Transport.prototype.normalizeQuery, Transport.prototype.formatResults, and common.clone were removed in winston 3.0.0, breaking query for File transport. To fix, examine the winston source code, particularly lib/winston/transports/transport.js and lib/winston/common.js, to understand the original query logic. Re implement query support for File transport by leveraging file system APIs like fs.createReadStream and manual result formatting. Ensure compatibility with the existing query API contract as documented. Testing should cover various query parameters (e.g., rows, limit, start, order).