search

mongo-express / mongo-express-docker

a dockerized mongo-express for viewing mongoDB in the browser
MIT License
201 stars 93 forks source link

Update entrypoint #101

Closed LaurentGoderre closed 9 months ago

LaurentGoderre commented 1 year ago

This is what I get with stock Node

docker run --rm -it --entrypoint bash node:20
root@2ca42459a867:/# cd /home/node/ 
root@2ca42459a867:/home/node# yarn policies set-version 4.0.0-rc.26
➤ YN0000: Retrieving https://repo.yarnpkg.com/4.0.0-rc.26/packages/yarnpkg-cli/bin/yarn.js
➤ YN0000: Saving the new release in .yarn/releases/yarn-4.0.0-rc.26.cjs
➤ YN0000: Done in 0s 860ms
root@2ca42459a867:/home/node# yarn add mongo-express@1.0.1
➤ YN0000: ┌ Resolution step
➤ YN0061: │ csurf@npm:1.11.0 is deprecated: Please use another csrf package
➤ YN0061: │ json2csv@npm:5.0.7 is deprecated: Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
➤ YN0001: │ Error: mongodb-query-parser@patch:mongodb-query-parser@npm%3A2.4.6#~/.yarn/patches/mongodb-query-parser-npm-2.4.6-234abad2fc.patch: ENOENT: no such file or directory, open '/home/node/.yarn/patches/mongodb-query-parser-npm-2.4.6-234abad2fc.patch'
➤ YN0000: └ Completed in 1s 497ms
➤ YN0000: Failed with errors in 1s 498ms
➤ YN0061: mongodb-extended-json@npm:1.11.1 is deprecated: Use EJSON from the `bson` package instead
root@2ca42459a867:/home/node#
LaurentGoderre commented 1 year ago

@BlackthornYugen I believe we need a release with this commit in: https://github.com/mongo-express/mongo-express/commit/5552eb5ecbe30cfa42fc3f1118c815728cecdca7

shakaran commented 9 months ago

@LaurentGoderre could you rebase conflicts? Thanks

LaurentGoderre commented 9 months ago

The PR in this state is not particularly useful but I can change it to get the version ready for release.

BlackthornYugen commented 9 months ago

It looks good to me but can someone give it a quick test?

LaurentGoderre commented 9 months ago

@BlackthornYugen I did test it locally and it worked

BlackthornYugen commented 9 months ago

@BlackthornYugen I did test it locally and it worked

I assumed you did, I just wanted to make sure it works on a different person's machine. Just extra cautious since the last few updates had issues. 😅

I can test in a few hours when I get home if nobody else is available.

BlackthornYugen commented 9 months ago

All good. Just wanted to be sure :)