CodeGenieApp / CodeGenieApp/serverless-express

req.path is incorrect when path ends with slash

Offen
#391 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
5.3k
Forks
676
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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`.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit zu ermitteln, wo req.path aus event.pathparameters.proxy abgeleitet wird, und vergleiche es mit event.path für Anfragen, die mit einem Schrägstrich enden. Füge einen Regressionstest hinzu, der /test und /test/ abdeckt, und überprüfe anschließend, dass der resultierende req.path den abschließenden Schrägstrich beibehält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
aws, express, javascript
Bereich
api, backend
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.