matthewmueller/x-ray

Uncaught ECONNRESET exception

Open

#171 ouverte le 9 avr. 2016

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (5 804 stars) (415 forks)batch import
bughelp wanted

Description

Subject of the issue

I found that the following apparently uncaught exception occurred and crashed our Node process instead of being caught by x-ray and forwarded as an err:

events.js:154
      throw er; // Unhandled 'error' event
      ^

Error: read ECONNRESET
    at exports._errnoException (util.js:890:11)
    at TCP.onread (net.js:552:26)

Your environment

node v5.9.1 npm v3.7.3

Steps to reproduce

Scrape from a source which resets the connection remotely.

Expected behaviour

Expected exception to be caught by x-ray and passed to callback as err.

Actual behaviour

Uncaught exception crashes process.

(PS many thanks for a great lib)

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
Node.js event handlingKnowledge of x ray library internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate the x ray library's HTTP request handling to find where ECONNRESET errors are not caught. Look in the main driver file (likely index.js or driver.js) for where requests are made. Ensure that the error event on the response stream is properly handled and forwarded to the callback. The issue suggests that the error is emitted on the socket and not caught by x ray's promise or callback structure.