db.streamChanges() seems broke about node-couchdb HOT 4 CLOSED

After a small amount of digging, I noticed that this wasn't working because streamChanges() sends its options through the toQuery() function, which JSON encodes the feed param.

Given that AFAIK the only JSON-valued parameters in CouchDB's API are key, startkey and endkey, I think you should instead whitelist the params to be JSON encoded:

function(query) {
for (var k in query) {
if (['key', 'startkey', 'endkey'].indexOf(k) != -1) {
query[k] = JSON.stringify(query[k]);
} else {
query[k] = String(query[k]);
}
}
return querystring.stringify(query);
}

I think this should cover your bases:
node> toQuery({ limit: 5, include_docs: true, key: { foo: "bar" }, stale: "ok" });
'limit=5&include_docs=true&key=%7B%22foo%22%3A%22bar%22%7D&stale=ok'

from node-couchdb.

Did this actually cause you any issues?

I ask because one of the couchdb guys told me all query values should be JSON quoted other than "stale=ok".

from node-couchdb.

This definitely wasn't working for me. Any value of feed besides continuous (w/o quotes) closes the HTTP connection immediately.

I've been writing code against my own branch just fine, with the parameter encoding patch described above. You should have a pull request waiting from me... (^_^)

from node-couchdb.

Thanks! Applied in d43a67a

from node-couchdb.