meteor-user-status icon indicating copy to clipboard operation
meteor-user-status copied to clipboard

If you're using this package along with Atlas Mongo 5.0, your project will not run on production environment

Open fidelsam1992 opened this issue 3 years ago • 13 comments

fidelsam1992 avatar Feb 17 '22 05:02 fidelsam1992

Since the forceful upgrade happened yesterday, if you are app is failing, try to remove this package

fidelsam1992 avatar Feb 17 '22 05:02 fidelsam1992

@StorytellerCZ @fidelsam1992 what needs to be done to make it run with Mongo 5 ?

jankapunkt avatar Feb 22 '22 12:02 jankapunkt

no solution found yet?

nick-preda avatar Apr 19 '22 20:04 nick-preda

@StorytellerCZ is there some action to be taken here? I am not using the package but I could take a look at it, depends on how severe the mongo 5 issue is.

jankapunkt avatar Apr 21 '22 06:04 jankapunkt

@jankapunkt if you could take a look that would be great. I'm unable to allocate any time for this till at least mid-May.

StorytellerCZ avatar Apr 21 '22 22:04 StorytellerCZ

I've actually attempted twice in the last week to get this on the latest version of meteor. For some reason every time I try to meteor update it says it fails and to check windows proxy settings... I might have to try on my mac to see if it's just a windows thing.

wreiske avatar Apr 22 '22 03:04 wreiske

@StorytellerCZ is there a reason for connection being explicitly set to null here: https://github.com/Meteor-Community-Packages/meteor-user-status/blob/master/server/status.js#L15

jankapunkt avatar Apr 25 '22 12:04 jankapunkt

@StorytellerCZ is there a reason for connection being explicitly set to null here: https://github.com/Meteor-Community-Packages/meteor-user-status/blob/master/server/status.js#L15

I double-checked and made sure it wasn't something I changed during the decaffeination. It looks like it was null while it was still caffinated: https://github.com/Meteor-Community-Packages/meteor-user-status/blob/b628f79d79077323dd16f4056bc7c66299ebee30/status.coffee#L8

From the docs: https://docs.meteor.com/api/collections.html#Mongo-Collection

Pass null to specify no connection.

Prior to that, this line showed: https://github.com/Meteor-Community-Packages/meteor-user-status/blame/a5312c58281d852f7932967872d0fbfff981fa76/status.coffee#L8

wreiske avatar Apr 25 '22 13:04 wreiske

Can someone please clone this package into your local packages folder and change that line to

 UserConnections = new Mongo.Collection("user_status_sessions") 

and then rebuild?

jankapunkt avatar Apr 25 '22 15:04 jankapunkt

If you go to the main branch of this repo, type "." and it will bring up an editor in the browser. You can edit it directly in github and then submit a PR. I believe the unit tests will run on the PR automatically. I have some time blocked tomorrow to look at this.

wreiske avatar Apr 25 '22 15:04 wreiske

thanks @wreiske I would avoid a PR until we find a solution that we can confirm to work

jankapunkt avatar Apr 25 '22 15:04 jankapunkt

TBH every time I go to update meteor this is always painful to dig through

C:\prj\meteor-user-status\demo>meteor update
This project is already at Meteor 2.7.1, the latest release.
=> Errors while upgrading packages:

While selecting package versions:
error: Conflict: Constraint [email protected] is not satisfied by accounts-base 2.2.2.
Constraints on package "accounts-base":
* accounts-base@~2.2.2 <- top level
* [email protected] <- mizzao:accounts-testing 0.1.0
* [email protected] <- service-configuration 1.3.0 <- accounts-base 2.2.2 <- accounts-ui 1.4.2 <- mizzao:accounts-testing 0.1.0
* [email protected] <- service-configuration 1.3.0 <- accounts-base 2.2.2 <- mizzao:accounts-testing 0.1.0
* [email protected] <- accounts-ui 1.4.2 <- mizzao:accounts-testing 0.1.0
* [email protected] <- accounts-ui-unstyled 1.7.0 <- accounts-ui 1.4.2 <- mizzao:accounts-testing 0.1.0
* [email protected] || 2.2.2 <- mizzao:user-status 1.0.2

I wish this could be easier!

Looks like we need to also update mizzao:accounts-testing

wreiske avatar Apr 27 '22 20:04 wreiske

Ok, made some progress today but I'm out of time https://github.com/Meteor-Community-Packages/meteor-user-status/pull/157

wreiske avatar Apr 27 '22 20:04 wreiske