possible bug in document_etag function about eve HOT 7 CLOSED

Excellent, thanks! Will review as soon as possible.

from eve.

And little addon - json.dumps not support bson.ObjectIt, but bson.json_util does
I think something like this will work:
from bson.json_util import dumps
dumps(value, sort_keys=True)

from eve.

Good call. However, have you actually seen the documents sorted differently upon restart/different server instances? I ask because I was under the impression that PyMongo/BSON documents use the SON class, which is basically a backport of Py3x/27 OrderedDict (I didn't verify it carefully though).

from eve.

Not working for me:

from bson.json_util import dumps

Traceback (most recent call last):
  File "<string>", line 1, in <fragment>
ImportError: cannot import name dumps

from eve.

@LarryEitel it should work:

from bson.json_util import dumps

def document_etag(value):
    h = hashlib.sha1()
    h.update(dumps(value, sort_keys=True))
    return h.hexdigest()

Passes all tests (well except test_document_etag of course, which is not updated yet)

from eve.

"However, have you actually seen the documents sorted differently upon restart/different server instances?"

Yes. I draw form, ajaxing data from form to eve server, and it pass till validation error (required field missing, in my case). I can do this several times with the same result.
Then, I Ctrl-C server and start it again (not closing form) and when I submit my form again, I ran into 412 error from server.

from eve.

Gotcha, will fix.

from eve.