fix: fix compatibility issue with MongoDB 4.X

[rawb1]

Could be a good idea to bump mongo version ? I did not find any information on mongo version compatibility ...

Fixes #1174

[changeset-bot]

🦋 Changeset detected

Latest commit: acc94fe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@accounts/mongo-password	Patch
@accounts/mongo-sessions	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codecov]

Codecov Report

Merging #1180 (acc94fe) into master (926b421) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1180   +/-   ##
=======================================
  Coverage   95.40%   95.40%           
=======================================
  Files         106      106           
  Lines        2351     2351           
  Branches      461      463    +2     
=======================================
  Hits         2243     2243           
  Misses        105      105           
  Partials        3        3           

Impacted Files	Coverage Δ
...ages/database-mongo-password/src/mongo-password.ts	100.00% <100.00%> (ø)
...ages/database-mongo-sessions/src/mongo-sessions.ts	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 926b421...acc94fe. Read the comment docs.

[pradel]

@rawb1 thanks for the fix! Instead of bumping mongo what do you think of keeping the old behavior, if the ops property is set and then fallback to your fix if not. That way we would support old and new versions and will not force our users to upgrade to the latest mongo.

[rawb1]

@pradel I think it is a great idea, I will update the PR tomorrow !

[rawb1]

@pradel I've done the changes but the other way around as it is easier to check if insertedId exist thant if ops[0]._id
Tell me if it is fine for you :)

I looked up the mongodb version the packages are using, I can look into doing the migration to v4 if you need !

@types/mongodb  ^3.6.20  →   ^4.0.6
mongodb          ^3.4.1  →   ^4.1.3

Btw great fan of the project, I used to be a big fan of meteor accounts package !

[pradel]

Migration to v4 would be amazing, we can release this one as a patch version and next pr in a separate minor version :)