PROBLEMA
Se desea obtener 2 valores enviados en una petición http
a una API desarrollada con Express. Los valores eerán usados para actualizar una tabla de MySQL mediante la iserción de un nuevo registro.
Entre los problemas que se observan en el código debo destacar que estás tratando de usar req.params
, pero la ruta que estás tratando de consumir no tiene ningún parámetro configurado:
/api/upload/
En tu pregunta dices también que has intentado pasar los valores en el cuerpo de la solicitud y has instalado y configurado una librería llamada body-parser
. Sin embargo, si utilizas ExpressJS v.4.16 o superior, no es necesario instalar dicha librería ya que la misma viene por defecto como parte de express
.
SOLUCIÓN
Suponiendo que deseas usar efectivamente parámetros en la URL que consumes, debes entonces crear el enrutamiento adecuado en la misma.
Por ejemplo:
router.put("/api/upload/:titulo/:descripcion", ...);
De esta manera le estoy indicando a Express que maneje la ruta de tal forma que el valor de título y de descripción son parte de la ruta, es decir son parámetros.
Como puedes imaginar, esto no es lo más adecuado para lo que intentas hacer, ya que por defecto los valores pasados por la URL serán codificados. Por ejemplo, si una descripción es muy larga: "Hola, esta es una nota de prueba", el resultado de la URL a consumir se verá así:
/api/upload/prueba/Hola%2C%20esta%20es%20una%20nota%20de%20prueba
Siendo el título: "Prueba" y la descripción: "Hola%2C%20esta%20es%20una%20nota%20de%20prueba"
Como se observa, tal vez no sea la mejor forma de enviar este tipo de datos al servidor.
Usando el body
o cuerpo de la solicitud
Lo mejor en estos casos es enviar la información en el cuerpo de la solicitud. Para ello podrías enviar los datos serializados en formato JSON.
Para poder leer los datos del cuerpo de la solicitud, Express necesita analizar dicho cuerpo. Aquí es donde entra en juego el parser o analizador del cuerpo. Express dispone de 2 métodos para lograr el objetivo:
express.json()
: es un middleware que analiza las solicutudes entrantes en busca de un Payload JSON.
express.urlencoded()
: es una función middleware que analiza las solicitudes entrantes y busca Pyloads codificados mediante urlencoding.
Sin entrar en muchos detalles de cómo funcionan ambos métodos, simplemente hago la sugerencia de que uses ambos como parte esencial de tu aplicación, recordando que debes usarlos antes de la declaración de cualquier ruta en la que sea necesario su uso. Por ejemplo, para tenerlo disponible en cualquier ruta en toda tu aplicación, puedes hacerlo de la siguiente forma:
const express = require('express');
const app = express();
app.use(express.json());
app.use(express.urlencoded({ extended: true }));
//...
app.use(routes ...);
Una vez que hemos usado el middleware para analizar el cuerpo de la solictud de forma correcta, podemos entonces pasar a la segunda parte del problema.
Actualizar sí y sólo si podemos hacerlo
Uno de los problemas con tu código es que da las cosas por sentadas. Es decir, falla simplemente porque en ningún momento te das a ala tarea de verificar si puedes insertar o no un nuevo registro.
Una de las reglas de oro en el desarrollo backend es: "NUNCA CONFÍES EN LO QUE ENVÍA EL CLIENTE".
Tu método para actualizar la tabla podría verse de la siguiente forma:
const insertNote = (req, res) => {
const { titulo, descripcion } = req.body;
if(!(titulo && descripcion)) {
return res.status(400).json({
message: "Datos insuficientes, falta titulo y/o descripcion",
status: 400
});
const preparedSQL = "INSERT INTO notas SET ?";
const values = { titulo, descripcion };
connection.query(preparedSQL, values, (err, result) => {
if(err) {
console.error('Uuuups');
console.error(error.message);
return res.status(500).json({
message: "Error al crear nota",
values,
status: 500
});
console.log(result);
return res.status(201).json({
message: "Nota creada"
result,
status: 201
});
}
})
}
En el código anterior se puede apreciar que en todo momento se envía una respuesta al cliente con return res.status()...
. De esta forma el cliente sabrá que la petición ha sido procesada por el servidor y la misma recibe una respuesta.
Lo primero que se hace es obtener los valores de título y descripción desde el cuerpo de la solicitud:
const { titulo, descripcion } = req.body;
Luego verificamos que los valores de título no sean cadenas vacías, null
ni undefined
. Si alguno de ellos o ambos es de este tipo, devolvemos un mensaje al cliente para indicarle que la solicitud no ha sido procesada porque hay un error en la misma. Es por ello que el estado de la respuesta se establece a 400 (Bad Request).
if(!(titulo && descripcion)) {
return res.status(400).json({ ... });
}
Por último, creamos la sentencia preparada para la inserción de los datos a la base de datos.
No sé cómo tienes configurada tu tabla, pero por lo general el valor de id
es auto numérico y auto generado por el motor de la base de datos. También el valor del campo completado
podría ser insertado automáticamente en una inserción mediante un valor por defecto. Pero en general sólo debes controlar que los valores pasados en el objeto que recibe el método query
son los justos y necesarios para poder llevar a cabo la operación sobre la base de datos.
En cuanto a la forma de enviar los datos desde el cliente al servidor, esto va a depender mucho del tipo de cliente que estés usando. En tu pregunta se muestra una imagen de lo que supongo que es POSTMAN. Por lo cual, enviar los datos en el cuerpo de la solicitud de la forma en que lo hace Postman es más que sucifiente.
Espero que esto te ayude a resolver el problema.
null
?app.use(express.urlencoded({ extended: true }) )
y ademásapp.use(express.json())
req.params
se consume de una forma específica. Puedes mostrar como manejas la ruta en Express y cómo envías la consulta desde el cliente?