search

t3db0t / meteor-accounts-invite

A login validation extension for Meteor that supports OAuth services
7 stars 1 forks source link

Invites blocking exist users from login... #4

Open laosb opened 8 years ago

laosb commented 8 years ago

For now I can't share a running instance, but it's easy to reproduce:

  1. meteor create xxx && cd xxx
  2. meteor add accounts-password accounts-ui
  3. Create a user.
  4. meteor add t3db0t:accounts-invite
  5. Try to login with the previously created user, reads User not found.

I'm not going to use this anymore. This package causes a lot of errors on server (though working) and even blocks myself from login. It's even easier to make one by hand (not supporting OAuth). Also, the temporary user is a user, causing all Meteor.user() Meteor.userId() checks failling as it does have a userId.

t3db0t commented 8 years ago

Thank you for the bug report—What are the other server errors you're getting?

Yes, the temporary user is a user, but for what it's worth, you can inspect the user's services.accountsInvite record for the data supplied in Meteor.loginWithInvite, which is what I do to differentiate temp users from 'real' users.

laosb commented 8 years ago

I removed this package and in my memory there are some errors like unexpected user undefined, but I can't promise what it actually looks like. Thank you for your hard work, though.