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

DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead. #6890

Closed
bricss opened this issue Aug 18, 2018 · 32 comments
Labels
enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature
Milestone

Comments

@bricss
Copy link

bricss commented Aug 18, 2018

After update to version 5.2.9 I'm getting this message in console all the time, when I start my web app:

DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.

And I don't use neither of them.

@Fonger
Copy link
Contributor

Fonger commented Aug 18, 2018

duplicate of #6880

@ghost
Copy link

ghost commented Aug 18, 2018

Thanks for taking the time to report the issue @bricss. Mongoose calls ensureIndex() internally. You can safely ignore the warning for now, or silence it with the node flag --no-deprecation, or downgrade to 5.2.8 if there are no specific fixes in 5.2.9 that you were relying on. Follow #6880 for updates 👍

@ghost ghost closed this as completed Aug 18, 2018
@nonniv
Copy link

nonniv commented Aug 23, 2018

,unique: true,index: true is on one field in one of my schemas and it triggers this warning. Question how to ensure a unique value without getting this warning.

@vkarpov15 vkarpov15 reopened this Aug 25, 2018
@vkarpov15 vkarpov15 added this to the 5.2.10 milestone Aug 25, 2018
@vkarpov15 vkarpov15 added the docs This issue is due to a mistake or omission in the mongoosejs.com documentation label Aug 25, 2018
@vkarpov15
Copy link
Collaborator

@nonniv upgrade to 5.2.10 and set mongoose.set('useCreateIndex', true);

@samuelcecilio
Copy link

samuelcecilio commented Aug 28, 2018

After upgrading to version 5.2.10. Any of the options below should stop the warnings?

mongoose.connect(config.dbUri, {
  useCreateIndex: true,
  useNewUrlParser: true
})

or

mongoose.set('useCreateIndex', true)
mongoose.connect(config.dbUri, { useNewUrlParser: true })

It didnt work here. (Sorry if I did something wrong)

@ghost
Copy link

ghost commented Aug 28, 2018

@samuelcecilio what specific deprecation warnings are you seeing? There are some helpful notes in the comment that opened #6922, or feel free to share your specific warnings here and I'll see if I can help.

@samuelcecilio
Copy link

(node:9125) DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.

@ghost
Copy link

ghost commented Aug 28, 2018

@samuelcecilio the mongoose.set('useCreateIndex', true) should stop that warning as demonstrated by this example:

6922.js

#!/usr/bin/env node
'use strict';

const mongoose = require('mongoose');
mongoose.connect('mongodb://localhost:27017/test', { useNewUrlParser: true });
mongoose.set('useCreateIndex', true);

const conn = mongoose.connection;
const Schema = mongoose.Schema;

const schema = new Schema({
  name: {
    type: String,
    unique: true
  }
});

const Test = mongoose.model('test', schema);

const test = new Test({ name: 'one' });

async function run() {
  console.log(`mongoose version: ${mongoose.version}`);
  await conn.dropDatabase();
  await test.save();
  return conn.close();
}

run();

Output:

issues: ./6922.js
mongoose version: 5.2.10
issues:

are you certain that your project is using 5.2.10? are there other packages in your project using other versions of mongoose, etc?

@samuelcecilio
Copy link

samuelcecilio commented Aug 28, 2018

Hi, thanks for the reply. Yes, I checked the version three times lol:

$ npm list --depth=0
...
 [email protected]
...

My code

app.js

...

const config = require('./config')
const mongoose = require('mongoose')
const express = require('express')
const app = express()

mongoose.connect(config.dbUri, { useNewUrlParser: true })
mongoose.set('useCreateIndex', true)

....

module.exports = app

bin/www

#!/usr/bin/env node

const app = require('../app')
const config = require('../config')
const debug = require('debug')('blitz')
const http = require('http')
const port = normalizePort(process.env.PORT || config.port)
const mongoose = require('mongoose')

app.set('port', port)

const server = http.createServer(app)

server.listen(port)
server.on('error', onError)
server.on('listening', onListening)

const mongo = mongoose.connection

mongo.on('error', error => { debug('mongo: ' + error.name) })
mongo.on('connected', () => { debug('mongo: Connected') })
mongo.on('disconnected', () => { debug('mongo: Disconnected') })

....

When I run DEBUG=blitz* npm run dev

> nodemon bin/www

[nodemon] 1.18.3
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: *.*
[nodemon] starting `node bin/www`
  blitz Listening on port 3000 +0ms
  blitz mongo: Connected +14ms
(node:10622) DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.

@ghost
Copy link

ghost commented Aug 28, 2018

@samuelcecilio maybe move the mongoose.set('useCreateIndex', true) to the bin/www file instead? I'll setup an express example and test as well.

@ghost
Copy link

ghost commented Aug 28, 2018

@samuelcecilio after playing around with this some, I found that I needed to set useCreateIndex to true in the route file that contained the model. In my case, after using the express cli tool, I needed to set useCreateIndex in the index route shown below:

var express = require('express');
var router = express.Router();
const mongoose = require('mongoose');
mongoose.set('useCreateIndex', true);
const Test = mongoose.model('test', { name: {type: String, unique: true }});

/* GET home page. */
router.get('/', function(req, res, next) {
  Test.create({ name: 'billy' }).then((doc) => {
    return res.status(200).json({ done: true, test: doc.id });
  }).catch((e) => {
    return res.status(500).json({ err: e });
  });
});

module.exports = router;

without the option set in the index route I got:

[nodemon] starting `node bin/www`
version: 5.2.10
(node:17042) DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.

after setting the option in the route I got:

[nodemon] starting `node bin/www`
version: 5.2.10
^C

I tried setting it in bin/www but that did not get rid of the warning. Only setting it in the route was successful.

output of curling localhost

~>: curl localhost:3000/
{"done":true,"test":"5b84a4e13ec72e4352475426"}~>: 
~>: 

@samuelcecilio
Copy link

I use separate files for users, customers, suppliers, orders, items..

This stopped the warning:

models/user.js

const addressSchema = require('./address')
const profileSchema = require('./profile')
const uuidv4 = require('uuid/v4')
const mongoose = require('mongoose')
const Schema = mongoose.Schema

mongoose.set('useCreateIndex', true)

const userSchema = new Schema({
  disabled: { type: Boolean, default: false },
  roles: { type: Array, default: [ 'assistant' ] },
  identity: { type: String, unique: true, required: true },
  username: { type: String, unique: true, required: true },
  name: { type: String, uppercase: true, required: true },
  password: { type: String, required: true },
  secret: String,
  profile: profileSchema,
  address: addressSchema,
  created: Date,
  modified: Date
}, {
  collection: 'users'
})

userSchema.pre('save', function (next) {
  const doc = this
  doc.created = Date.now()
  doc.modified = Date.now()
  doc.secret = uuidv4()
  next()
})

module.exports = mongoose.model('User', userSchema)

I figured it would be some global setting, but apparently I have to add in all the files.

@ghost
Copy link

ghost commented Aug 28, 2018

@samuelcecilio you could wrap mongoose in a custom file, something like

myMongoose.js

const mongoose = require('mongoose')
mongoose.set('useCreateIndex', true);
//etc
module.exports = mongoose

and then require your wrapper const mongoose = require('myMongoose') instead of mongoose everywhere. That's probably something like what I'll end up having to do when I start upgrading beyond 5.2.8.

@samuelcecilio
Copy link

Still I would have to modify all current files. For each file in the models folder, I would replace require ('mongoose') with require('myMongoose'). The same would apply to app.js and bin/www.

The question is: in a future version of mongoose I will not need to include mongoose.set ('useCreateIndex', true) or will this be normal, for compatibility issues with older versions of mongo, node, etc ...?

@ghost
Copy link

ghost commented Aug 28, 2018

I suspect the behavior will remain as it is now ( having to set the variable, not the internal implementation of it ) until the next major version (ie 6.0.0 ) of mongoose.

@samuelcecilio
Copy link

For now, I've added mongoose.set('useCreateIndex', true) to all files that contain require('mongoose'). I'm thinking of using a wrapper ('./myMongoose') or ('./mongoose-custom'). What do you think?

Should this be global? Is it an express issue?

@bricss
Copy link
Author

bricss commented Aug 28, 2018

It's really uncomfortable to add mongoose.set('useCreateIndex', true) in every single file that contain model declarations.
Why is not make it work from connect options as well, or set it before connection?

@ghost ghost reopened this Aug 28, 2018
@ghost
Copy link

ghost commented Aug 28, 2018

@samuelcecilio @bricss Agreed. Reopening this to investigate potential solutions.

@vkarpov15
Copy link
Collaborator

You don't need to put this in every file, but unfortunately it looks like right now you need to put mongoose.set('useCreateIndex', true) before any mongoose.model() calls. That is tricky if you export models as opposed to schemas. Will add a fix for this.

@vkarpov15 vkarpov15 modified the milestones: 5.2.10, 5.2.11 Aug 29, 2018
@vkarpov15 vkarpov15 added enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature and removed docs This issue is due to a mistake or omission in the mongoosejs.com documentation labels Aug 29, 2018
@vkarpov15 vkarpov15 removed this from the 5.2.11 milestone Aug 30, 2018
@samuelcecilio
Copy link

After upgrading to version 5.2.13, should this stop the warnings? Thanks

mongoose.connect(config.dbUri, {
  useCreateIndex: true,
  useNewUrlParser: true
})
the options [useCreateIndex] is not supported
server: listening on port 3000 +0ms
mongo: Connected +34ms
(node:18457) DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.

@ghost
Copy link

ghost commented Sep 5, 2018

@samuelcecilio useCreateIndex isn't a connection option, but as long as you call mongoose.set('useCreateIndex', true); before calling mongoose.model() ( even in other files ) all subsequent calls to mongoose.model should fallback to the option you set with mongoose.set(). I tested 5.2.13 with my example above, instead calling mongoose.set('useCreateIndex', true) in the bin/www file and not in my route. It works now.

@aclarturocastro
Copy link

It works with 5.2.13 after connection and before all other files.

This is connection in my db.js file:

// Create the database connection
mongoose.connect(encodeMongoURI(dbURI), options);
mongoose.set('useCreateIndex', true);

@LucHighwalker
Copy link

I have my mongo connection in a separate script. This way I just call mongoose.set('useCreateIndex', true); right before actually connecting to mongo. Then there's no need to put it in any of my model declarations.

@vkarpov15
Copy link
Collaborator

@LucHighwalker 's suggestion is the right way to do this. Ideally you put all these global options right before your call to mongoose.connect().

@cloakedch
Copy link

Is there any functional difference between

mongoose.connect(encodeMongoURI(dbURI), options);
mongoose.set('useCreateIndex', true);

and

mongoose.set('useCreateIndex', true);
mongoose.connect(encodeMongoURI(dbURI), options);

? Because both seem to work just fine in 5.2.17

@samuelcecilio
Copy link

mongoose.connect(config.dbUri, {
  useCreateIndex: true,
  useNewUrlParser: true
})

Works with the latest version

@cloakedch
Copy link

cloakedch commented Sep 23, 2018

Okay, i guess @types/mongoose is some versions behind then (doesn't work on typescript yet).

@DionesioJr
Copy link

mongoose.connect(url, { useNewUrlParser: true, autoIndex: false })

@jrmatos
Copy link

jrmatos commented Sep 25, 2018

@Dionesio's answer did the trick to me. Thanks!

@darshanan24
Copy link

mongoose.connect("mongodb://localhost:27017/test", {useNewUrlParser: true});
mongoose.set('useCreateIndex', true);

This solved all my deprecation warnings.

@amolbhandari52
Copy link

@samuelcecilio after playing around with this some, I found that I needed to set useCreateIndex to true in the route file that contained the model. In my case, after using the express cli tool, I needed to set useCreateIndex in the index route shown below:

var express = require('express');
var router = express.Router();
const mongoose = require('mongoose');
mongoose.set('useCreateIndex', true);
const Test = mongoose.model('test', { name: {type: String, unique: true }});

/* GET home page. */
router.get('/', function(req, res, next) {
  Test.create({ name: 'billy' }).then((doc) => {
    return res.status(200).json({ done: true, test: doc.id });
  }).catch((e) => {
    return res.status(500).json({ err: e });
  });
});

module.exports = router;

without the option set in the index route I got:

[nodemon] starting `node bin/www`
version: 5.2.10
(node:17042) DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.

after setting the option in the route I got:

[nodemon] starting `node bin/www`
version: 5.2.10
^C

I tried setting it in bin/www but that did not get rid of the warning. Only setting it in the route was successful.

output of curling localhost

~>: curl localhost:3000/
{"done":true,"test":"5b84a4e13ec72e4352475426"}~>: 
~>: 

Thanks - Adding mongoose.set('useCreateIndex', true); fixed my issue

@vkarpov15
Copy link
Collaborator

I'm going to lock this issue. For future reference, see the deprecations guide on our docs

@Automattic Automattic locked as resolved and limited conversation to collaborators Dec 6, 2018
Paaaaak referenced this issue in Paaaaak/mern-project-mapping Sep 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature
Projects
None yet
Development

No branches or pull requests