Sale ya

[leoAchu16]

Summary by CodeRabbit

Notas de Lanzamiento

• Chores
  • Ajustados límites de intentos de inicio de sesión (aumentado a 10 intentos por 15 minutos).
  • Aumentados límites de solicitud de restablecimiento de contraseña (ahora 5 solicitudes por hora).
  • Actualizada configuración de proxy para mejor compatibilidad en entornos con servidores upstream.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Se agrega configuración de proxy upstream en Express para interpretar correctamente direcciones IP detrás de proxies, y se incrementan los límites de rate limiting para autenticación (5→10 intentos) y recuperación de contraseña (3→5 solicitudes) por período ventana.

Changes

Configuración de proxy y rate limiting

Layer / File(s)	Resumen
Configuración de trust proxy src/app.js	Se introduce app.set('trust proxy', 1) antes de los middlewares de seguridad para delegar la confianza al primer proxy intermedio, ajustando cómo Express interpreta req.ip al estar detrás de proxies como Render, Heroku o Nginx.
Ajuste de límites de rate limiting src/middlewares/rateLimiter.js	loginRateLimiter aumenta de 5 a 10 intentos por IP cada 15 minutos, y passwordResetRateLimiter aumenta de 3 a 5 solicitudes por IP cada hora, ambos con comentarios actualizados.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutos

Possibly related PRs

• CrisNAC/BackendMarketplace#194: Introduce los middlewares loginRateLimiter y passwordResetRateLimiter que son ajustados en este PR para incrementar sus límites de velocidad.

Poem

🐰 Conejo saltarín, velocidad a ajustar,
Proxys de confianza y límites a modular,
Diez intentos ahora, cinco para resetear,
La configuración lista para el servidor volar. 🚀

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	El título 'Sale ya' es vago y genérico, no describe de manera clara o específica los cambios realizados en el pull request.	Reemplazar el título con una descripción clara de los cambios principales, como 'Ajustar configuración de proxy y límites de rate limiting' o similar.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch OM-554

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud