Conexión de nuestros servicios con MongoDB

Clase 29 de 48 • Curso de Backend con Node.js 2019

Carlos Vallejo

student•

Si tienen el siguiente error:

MongoError: MongoClient must be connected before calling

Traten con esto:

Quiten el número de puerto (${config.port}) del string de conexión en lib/mongo.js, debería quedar así:

const MONGO_URI = `mongodb+srv://${USER}:${PASSWORD}@${config.dbHost}/${DB_NAME}?retryWrites=true&amp;w=majority`;

En el mismo archivo lib/mongo.js aumentar un parámetro al constructor del this.client. Debería quedar así:

this.client = new MongoClient(MONGO_URI, { useNewUrlParser: true, useUnifiedTopology: true });

Espero esto les sirva.

Carlos Enrique Ramírez Flores

student•

no manches! te debo una. Tengo el mismo error. muchas gracias de verdad!

Juan Jose Vega

student•

Alguien me podría ayudar Mis variables de entorno me retornan undefined :(

Alejandro González Reyes

student•

Con la configuración actual envía un warning

DeprecationWarning: current Server Discovery and Monitoring engine is deprecated, and will be removed in a future version. To use the new Server Discover and Monitoring engine, pass option { useUnifiedTopology: true } to the MongoClient constructor

Añadir lo siguiente al archivo de librería "MongoLib"

constructor() {
    this.client = new MongoClient(MONGO_URI, {
      useNewUrlParser: true,
      useUnifiedTopology: true
    });
    this.dbName = DB_NAME;
  }

Andrea Méndez Díaz

student•

Gracias.

Jesús Miguel Quinto Teran

student•

Thanks

Jesús Miguel Quinto Teran

student•

A diferencia de los JSON que escribimos comúnmente, para que express logre parcear correctamente el json, deben usar comillas dobles en la clave.


//bad 💥
{
	title: "lorem ipsum"
}

// bad 💥
{
	'title': 'lorem ipsum'
}


//Good ✨
{
	"title": "lorem ipsum"'
}

Javier Armando Vargas Vega

student•

Muchas gracias por el aporte.

Jhony Alexander Garcia Gómez

student•

Para la solución del reto de tener diferentes bases de datos dependiendo de si estamos en desarrollo o en producción, encontré una libreria en npmjs que se llama dotenv-flow que permite manejar estas configuraciones.

Instalar el paquete

npm install dotenv-flow --save

Creé un nuevo archivo de configuracion .env.development de copia del .env y le cambie los parametros
En el archivo /config/index.js cambié la libreria a usar por dotenv-flow

require('dotenv-flow').config();

Agregue al .gitignore el archivo .env.development para que no sea cargada mis credenciales de desarrollo al repositorio

# dotenv environment variables file
.env
.env.test
.env.development

En el package.json modifiqué el script dev agregando a la variable de NODE_ENV=development

"scripts": {
  "dev": "DEBUG=app:* NODE_ENV=development nodemon index",
  "start": "NODE_ENV=production node index"
},

Con estos cambios logré separar las configuraciones del ambiente de producción al ambiente de desarrollo. El paquete de dotenv-flow se encarga basado en NODE_ENV cargar el archivo de configuración respectivo. Este paquete tiene más opciones, dejo el enlace del repositorio dotenv-flow

Erick Avila Rojas

student•

Gracias viejo. Es la mejor solución que he visto aquí; yo solo había hecho el cambio de la instancia directamente en el archivo env

Wandy Rafael Santana Evangelista

student•

Hola Devs: Aqui les traigo las correcion a varias parte del codigo del profesor, tal vez les ayuden a corregir sus errores:

Uno de ellos esta en la URI de Mongo, el profesor la tiene de estas manera:

const MONGO_URI = `mongodb+srv//${USER}:${PASSWORD}@${config.dbHost}:${config.port}/${DB_NAME}?retryWrites=true&w=majority`;

Y nosotros deberiamos borrar el PORT y dejarlo como nos indica Mongo que deberiamos hacer la conexion, de esta manera:

const MONGO_URI = `mongodb+srv://${USER}:${PASSWORD}@${config.dbHost}/${DB_NAME}?retryWrites=true&w=majority`;

Otro Error seria en nuestro archivo movies.js en nuestra carpeta de servicios el metodo update tiene un error, el profesor llamo mal el metodo para actualizar, el profesor lo tiene de esta manera:

const updateMovieId = await this.mongoDB.create(this.collection, movieId, movie);

Y nosotros deberiamos cambiarlo al metodo update, de esta manera:

const updateMovieId = await this.mongoDB.update(this.collection, movieId, movie);

Y un plus o extra seria agregar un parametro mas que nos indica mongo a nuestra instancia de mongo client:

this.client = new MongoClient(MONGO_URI, { useNewUrlParser: true , useUnifiedTopology: true,});

Espero que corregir estos detalles ayude a que puedan resolver con exitos sus errores, recuerden #NuncaParesDeAprender :)

Andres Roberto Coello Goyes

student•

En mi caso me arrojaba un error del PORT, y me di cuenta que en la conexión del cluster

mongodb+srv://<username>:<password>@cluster0-pm1ma.mongodb.net/test?retryWrites=true&w=majority

no especificaba el puerto así que lo borre y listo.

Jorge Alonso Molina Chirinos

student•

muchisimas gracias me funcionó perfecto 🙌🙌🙌

Marco Mesén Campos

student•

Algún heroe sin capa que explique esta linea a detalle:

const query = tags &amp;&amp; {tags: {$in: tags}};

Carlos Vallejo

student•

Esto es lo que se llama un IF Ternario, cumple la misma función de un condicional IF, nada más que escrito de distinta manera.

const ifBasico = &lt;condición&gt; &amp;&amp; &lt;haz esto&gt;;
const ifConElse = &lt;condición&gt; &amp;&amp; &lt;haz esto&gt; : &lt;sino haz esto&gt;;

En el caso del profesor, lo que hace es una verificación para ver si la variable tags está definida (hay datos) y en caso de que así sea está creando un objeto con el atributo tags

Javier Armando Vargas Vega

student•

Gracias Carlos por la explicación. Tenía la misma duda

David Behar

student•

Para conectarme a mi base de datos tuve que modificar mi endpoint de la siguiente manera

const MONGO_URI = `${config.dbConnectionType}://${USER}:${PASSWORD}@${config.dbHost}:${config.dbPort}`;

Donde dbConnectionType cambia a mongodb, también tuve que quitar la base de datos porque tenemos el código

          resolve(this.client.db(this.dbName));

En ese código realizamos de nuevo la conección a una base de datos específica, si alguien batalla por conectarse a base de datos privada, le ayudaré desde este mensaje.

Javier Ramírez

student•

Me da un error al hacer el create:

TypeError: Cannot read property '_id' of undefined

Javier Ramírez

student•

Solucionado: era unas llaves que deberían ir en el createMovie({movie})

Hector Romero

student•

Ahora yo tengo ese error y no se como solucionarlo

	async createMovie({ movie }) {
		const createdMovieId = await this.mongoDB.create(this.collection, movie);
		return createdMovieId;
	}

Asi es como lo tengo

Jonatan Francisco Gonzalez Donis

student•

Una duda ahi dejas la conexión abierta de mongo. Si utilizo un driver para una base de datos relacional ejemplo oracle-db. Es recomendable igual dejar indefinidamente la conexión abierta??

Guillermo Rodas

teacher•

La verdad depende el tipo de de aplicación que este usando. Por ejemplo, cuando es una aplicación que tiene muchas transacciones lo mejor es dejar las conexiones abiertas para que se reutilicen, porque es mas costoso abrir y cerrar conexiones. Pero generalmente, mantener una conexión abierta sin transacciones es un consumo de memoria no necesario.

Los drivers suelen tomar esta decisión por ti. Mi consejo, es ensaya varias opciones y monitorea como se comporta tu aplicación para tomar la mejor decisión.

Humberto OTÁLORA

student•

hay un salto en el cuar se cambia el scrip de deleteMovie

Joaquín Ignacio Ossandón Gómez

student•

Hasta aquí, si coloco un solo tag en la query, no busca y manda error, porque $in necesita que "tags" sea un array, mi solución fue convertir tags en array si es que es un string.

  async getMovies({ tags }) {

    typeof(tags) == 'string' ? 
	tags = [tags] : //si es string lo paso como array
	 tags

    const query = tags && { tags: { $in: tags } }
    console.log(query)
    const movies = await this.mongoDB.getAll(this.collection, query)
    return movies || []
  }

Seguro más adelante hay una solución más elegante.

también dejo la referencia de operadores de comparación ($in) https://docs.mongodb.com/manual/reference/operator/query-comparison/

Juan Esteban Galvis

student•

Me salvaste jaja llevaba mero rato buscando jajaja gracias héroe sin capa

Vicente Daniel Zurita Salas

student•

Si les arroja error prueben quitando el puerto, ya que en la cadena de conexión del cluster no se incluye el puerto.

const MONGO_URI = `mongodb+srv://${USER}:${PASSWORD}@${config.dbHost}/${DB_NAME}?retryWrites=true&w=majority`;

además que en config tenemos port, no dbport.

Nadir Antonio Soza Solis

student•

Hola @glrodasz tu recomendarías tener una única clase Servicio que reciba en su constructor el nombre de la colleccion y que esta implemente los CRUD's para todas? algo similar a esto...

const MongoLib = require('../lib/mongo');

class ServicesCRUD {
  constructor(_collection) {
    this.collection = _collection;
    this.mongoDB = new MongoLib();
  }

  async getDataByTags({ tags }) {
    const query = tags && { tags: { $in: tags } };
    const data = await this.mongoDB.getAll(this.collection, query);
    return data || [];
  }

  async getDataById({ ID }) {
    const data = await this.mongoDB.get(this.collection, ID);
    return data || [];
  }

  async createData({ data }) {
    const createdDataId = await this.mongoDB.create(this.collection, data);
    return createdDataId;
  }

  async updateDataById({ ID, data } = {}) {
    const updatedDataId = await this.mongoDB.update(this.collection, ID, data);
    return updatedDataId;
  }

  async deleteDataById({ ID }) {
    const deletedDataId = await this.mongoDB.delete(this.collection, ID);
    return deletedDataId;
  }
}

module.exports = ServicesCRUD;

Jesús Miguel Quinto Teran

student•

@glrodasz Me hago la misma pregunta, realizar CRUD es tarea de todos los días y podría ser muy bueno este enfoque. ¿tu que opinas?

Hector Romero

student•

Creo que estoy tambien de acuerdo con esta solucion, pero que tan viable es a largo plazo?

Camilo Morales Sanchez

student•

buenas noches, la verdad estoy perdido y no entendí el challenge propuesto. Estoy algo aburrido porque no he podido entender casi nada :(

Christian Perez

student•

No es ayuda, pero yo estoy igual jaja

Camilo Morales Sanchez

student•

jajajaja :(

David Lara

student•

Casi 2 horas leyendo documentacion y repitiendo videos, pero todo bien. Por fin!

Carlos Daniel

student•

¿En qué casos sería idóneo cerrar la conexión a base de datos y en qué parte del código? Quiero decir, no veo que en ninguna parte del módulo se cierre nunca la conexión a base de datos por lo que asumo que no debe de consumir muchos recursos mantenerla abierta para esta aplicación, además de que, si quisiéramos cerrarla ¿lo haríamos dentro de la misma clase que la expone? pero de hacerlo así si múltiples usuarios se conectan ¿Eso no abrirá una conexión por cada usuario y hará que la DB se sobrecargue de conexiones?

Massimo Di Berardino

student•

Hola @winterowl. No, no se abre una conexión por usuario. Lo de cerrar la conexión y abrirla "on-demand" es más un concepto de serverless, puedes profundizar sobre ese tema acá

Juan Burns

student•

Buenas noches!, porque agrega = { } en updateMovie en el minuto 3:08, dice que para no tener problemas, pero no explica porque tendria problemas, por favor, muchas cosas explicadas a medias, saludos!.

Erik Elyager

student•

Le asigna un objeto vacío al parámetro para que en caso de que no sea enviado al invocar la función no tengamos un problema de valor undefined al querer accesar ese dato.

Daniel Sánchez

teacher•

Repositorio de la clase: https://github.com/glrodasz/platzi-backend-node/tree/capa-de-manejo-de-errores-usando-un-middleware

Cristian Daniel Jesus Rios

student•

Buenas, si se estan volviendo locos con la conexión o les tira un error el postman similar a este "Error: connect ECONNREFUSED", vayan a mongo atlas - Clusters - Connect y se les abre un modal, tocan la opción del medio "Connect your application", una vez dentro copian el trozo de código que les da el recuadro con los datos de su conexión y lo pegan dentro de su código mongo.js que hicieron en el video

const MONGO_URI = `mongodb+srv://db_user_platzivideo:${PASSWORD}@cluster0.lzplb.mongodb.net/${DB_NAME}?retryWrites=true&w=majority`;