CodeGenieApp / CodeGenieApp/serverless-express

req.path is incorrect when path ends with slash

Abierto
#391 3 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
JavaScript
Estrellas
5.3k
Forks
676
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

req.path is set like below.

```
1. https://domain/test => req.path = '/test'
2. https://domain/test/ => req.path = '/test'
```

2 is not correct, since Express actually set '/test/' to path in this case.

I think the cause is that the `req.path` is obtained from `event.pathparameters.proxy` instead of `event.path`, and the last slash is not included in `event.pathparameters.proxy`.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza localizando dónde se deriva req.path a partir de event.pathparameters.proxy y compáralo con event.path para las solicitudes que terminan en una barra. Añade un caso de regresión que cubra /test y /test/ y, a continuación, verifica que el req.path resultante conserve la barra final.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
aws, express, javascript
Área
api, backend
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.