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

Dokku-Mongo uses deprecated mongo instead of mongosh #156

Closed
TimHal opened this issue Aug 25, 2022 · 2 comments · Fixed by #166
Closed

Dokku-Mongo uses deprecated mongo instead of mongosh #156

TimHal opened this issue Aug 25, 2022 · 2 comments · Fixed by #166

Comments

@TimHal
Copy link

TimHal commented Aug 25, 2022

Description of problem

The plugin appears to use mongo which is deprecated in favor of the newer mongosh.

How reproducible

Install the plugin.
Connect as admin via
dokku mongo:connect-admin <service>

Steps to Reproduce

  1. Install the plugin
  2. dokku mongo:connect-admin <service>

Actual Results

A mongo-based connection with the warning

================
Warning: the "mongo" shell has been superseded by "mongosh",
which delivers improved usability and compatibility.The "mongo" shell has been deprecated and will be removed in
an upcoming release.
For installation instructions, see
https://docs.mongodb.com/mongodb-shell/install/

Expected Results

A mongosh-based connection

dokku report APP_NAME output

No app required

How (deb/make/rpm) and where (AWS, VirtualBox, physical, etc.) was Dokku installed?:

Following the dokku getting started guide

@josegonzalez
Copy link
Member

Is the mongo shell built into the official image?

@TimHal
Copy link
Author

TimHal commented Aug 25, 2022

Yes, mongo is still kept for backwards compatibility I think but anything from version 5.x+ can use mongosh.

See https://hub.docker.com/_/mongo (Chapter "Connect to MongoDB from another Docker container")

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

Successfully merging a pull request may close this issue.

2 participants