Winston: कस्टम परिवहन उदाहरण में सुधार करें

को निर्मित 4 जुल॰ 2018 · 3टिप्पणियाँ · स्रोत: winstonjs/winston

कस्टम ट्रांसपोर्ट कोड उदाहरण जोड़ने में, log विधि में, callback और setImmediate कॉल दोनों के उद्देश्य पर कोई स्पष्टीकरण नहीं है।

log(info, callback) {
    // what is this for? Does Winston need it?
    setImmediate(() => {
      this.emit('logged', info);
    });

    // Perform the writing to the remote service
    callback(); // is it required to include this cb in all transports? Does Winston need it?
  }

स्रोत

Buzut

👍24

सबसे उपयोगी टिप्पणी

मैं सहमत हूं। मुझे यह भी लगता है कि उदाहरण में सुधार किया जा सकता है। टिप्पणी // Perform the writing to the remote service को ध्यान में रखते हुए मुझे लगता है कि दूरस्थ सेवा एक एसिंक कॉल है। अन्य कस्टम ट्रांसपोर्ट को देखने के बाद, क्या उदाहरण को इस तरह नहीं पढ़ना चाहिए?

log(info, callback = () => {}) {
    callRemoteService(info)
        .then(result => {
            callback(null, result);
            this.emit('logged', info);
        })
        .catch(error => {
            callback(error);
            this.emit('error', error);
        })
}

कॉलबैक के लिए डिफ़ॉल्ट मान मुझे transports.File कोड में मिला।

davidbeermann 20 जुल॰ 2018

👍6 🚀1

सभी 3 टिप्पणियाँ

log(info, callback = () => {}) {
    callRemoteService(info)
        .then(result => {
            callback(null, result);
            this.emit('logged', info);
        })
        .catch(error => {
            callback(error);
            this.emit('error', error);
        })
}

कॉलबैक के लिए डिफ़ॉल्ट मान मुझे transports.File कोड में मिला।

davidbeermann 20 जुल॰ 2018

👍6 🚀1

यह वास्तव में मददगार लगेगा। यहां उदाहरण सीमित है जैसा कि https://github.com/winstonjs/winston-transport रिपॉजिटरी में उदाहरण है।

कई कस्टम परिवहन के माध्यम से देख रहे हैं और कोई भी दो समान नहीं हैं।