CodeGenieApp / CodeGenieApp/serverless-express
`body` data not passed correctly to the express app
- Vorherrschende Sprache
- JavaScript
- Sterne
- 5.3k
- Forks
- 676
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Currently, when using express, the `req.body` data is incorrectly passed to the express application. I am getting the gibberish trash data from somewhere passed into the request. This problem does not exists if I use `getCurrentInvoke` to get the `body` data, which will be correct.
Propose to fix this by changing this line here https://github.com/vendia/serverless-express/blob/mainline/src/frameworks/express.js#L2 from `app.handle` to `app`, which should solve the issue. See https://github.com/expressjs/express/blob/master/lib/express.js#L38.
Currently I am using express v4, not sure if this will be the case for express v5 or even prior versions.
I will be more than happy to work on this issue (and other issues as a contributor).
Maybe we can make the `event` & `context` data not available for the express `Request` object in the future as well for convenience.
Beitragsleitfaden
Rechercherichtung
Beginne bei src/frameworks/express.js Zeile 2 und vergleiche die aktuelle app.handle-Referenz mit Express's lib/express.js Zeile 38. Überprüfe, dass Express 4 req.body mit dem über getCurrentInvoke erhaltenen Body übereinstimmt, und prüfe das Verhalten anhand der angegebenen Unsicherheit bezüglich der Express-Version.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- express, javascript, node.js
- Bereich
- api, backend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 45/100