Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Achieve compatibility with Meteor examples #2414

Open
9 of 13 tasks
Tracked by #5
AlekSi opened this issue Apr 12, 2023 · 5 comments
Open
9 of 13 tasks
Tracked by #5

Achieve compatibility with Meteor examples #2414

AlekSi opened this issue Apr 12, 2023 · 5 comments

Comments

@AlekSi
Copy link
Member

AlekSi commented Apr 12, 2023

Test https://github.com/meteor/examples with FerretDB, create and link issues there.
Check that those examples use OpLog.

Examples

Issues

  1. area/indexes code/feature not ready
    ferretdb-bot
  2. area/indexes code/feature
    rumyantseva
  3. code/bug
    w84thesun
  4. code/feature
    chilagrow
  5. badly estimated code/enhancement
    chilagrow

See also:

@AlekSi AlekSi added the code/feature Some user-visible feature is not implemented yet label Apr 12, 2023
@AlekSi AlekSi assigned b1ron and unassigned ferretdb-bot Apr 12, 2023
@b1ron
Copy link
Contributor

b1ron commented Apr 12, 2023

Sounds good!

@AlekSi AlekSi added the not ready Issues that are not ready to be worked on; PRs that should skip CI label May 1, 2023
@b1ron
Copy link
Contributor

b1ron commented May 1, 2023

We are basically compatible with the blaze tutorial if we exclude #2448. This tutorial is what https://github.com/wreiske/meteor-ferretdb is pulling from.

To test it I ignored sparse indexes and it worked.

It doesn't seem to be using the $ projection operator to refresh user tokens, so I don't get logged out of the session. But I still see the below error:

➜  step13 git:(master) ✗ MONGO_URL=mongodb://localhost:27017/ meteor
[[[[[ ~/workdir/blaze-tutorial/src/simple-todos/step13 ]]]]]

=> Started proxy.                             
=> Started HMR server.                        
=> Meteor 2.11.0 is available. Check the changelog https://docs.meteor.com/changelog.html and update this project with 'meteor update'.
=> Started your app.

=> App running at: http://localhost:3000/
I20230501-20:58:48.854(2)? Exception while invoking method 'login' TypeError: Cannot read property 'resume' of undefined
I20230501-20:58:48.881(2)?     at MethodInvocation.defaultResumeLoginHandler (packages/accounts-base/accounts_server.js:1577:29)
I20230501-20:58:48.881(2)?     at MethodInvocation.<anonymous> (packages/accounts-base/accounts_server.js:1532:38)
I20230501-20:58:48.882(2)?     at packages/accounts-base/accounts_server.js:594:31
I20230501-20:58:48.882(2)?     at tryLoginMethod (packages/accounts-base/accounts_server.js:1518:14)
I20230501-20:58:48.882(2)?     at AccountsServer._runLoginHandlers (packages/accounts-base/accounts_server.js:592:22)
I20230501-20:58:48.882(2)?     at MethodInvocation.methods.login (packages/accounts-base/accounts_server.js:652:31)
I20230501-20:58:48.882(2)?     at maybeAuditArgumentChecks (packages/ddp-server/livedata_server.js:1885:12)
I20230501-20:58:48.882(2)?     at packages/ddp-server/livedata_server.js:769:19
I20230501-20:58:48.882(2)?     at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)
I20230501-20:58:48.883(2)?     at packages/ddp-server/livedata_server.js:767:46
I20230501-20:58:48.883(2)?     at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)
I20230501-20:58:48.883(2)?     at packages/ddp-server/livedata_server.js:765:46
I20230501-20:58:48.883(2)?     at new Promise (<anonymous>)
I20230501-20:58:48.883(2)?     at Session.method (packages/ddp-server/livedata_server.js:739:23)
I20230501-20:58:48.885(2)?     at packages/ddp-server/livedata_server.js:603:43

@AlekSi AlekSi removed not ready Issues that are not ready to be worked on; PRs that should skip CI code/feature Some user-visible feature is not implemented yet labels May 2, 2023
@AlekSi AlekSi mentioned this issue May 11, 2023
@mitar
Copy link

mitar commented May 11, 2023

If you really want to stress test out Meteor and MongoDB integration, I suggest using my old meteor-peerdb test suite. It has managed to find issues with Meteor implementation itself. :-)

@b1ron
Copy link
Contributor

b1ron commented May 11, 2023

thanks @mitar, we will check it out!

@AlekSi
Copy link
Member Author

AlekSi commented Jan 8, 2024

We just released a new version with OpLog tailing support, which was a major blocker for supporting Meteor apps: https://github.com/FerretDB/FerretDB/releases/tag/v1.18.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants