felixge / felixge/node-couchdb

HTTP error detection

Offen
#49 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
363
Forks
68
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The code (near [line 232](https://github.com/felixge/node-couchdb/blob/master/lib/couchdb.js#L232)) that detects CouchDB HTTP errors seems dangerous to me:

``` javascript
if ('error' in json) {
cbFired = true;
return cb && cb(json);
}
```

If I save a document with an 'error' attribute, node-couchdb will raise an error every time I fetch it. You should rely on the HTTP status code instead.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.