Main reason to do this PR is that insert proc always failed on response["n"] conversion. This is because value of response["n"] is BsonKindDouble. nimongotest.nim failed too.
To fix it I have unified database reply parsing. This should fix several issues like in insert but not break backwards compatibility. This also adds errmsg to ReplyStatus if error occurs in all cases.
Tested with
nim --version
Nim Compiler Version 0.15.1 (2016-10-15) [Linux: amd64]
git hash: 2d2b1a9d481bffaecac35e1e52929cea66f69e0e
mongod --version
db version v3.3.15
git version: 520f5571d039b57cf9c319b49654909828971073
Main reason to do this PR is that
insertproc always failed onresponse["n"]conversion. This is because value ofresponse["n"]isBsonKindDouble.nimongotest.nimfailed too.To fix it I have unified database reply parsing. This should fix several issues like in
insertbut not break backwards compatibility. This also addserrmsgtoReplyStatusif error occurs in all cases.Tested with
nimongotest.nimgreen (fixed and updated it).