CodeGenieApp / CodeGenieApp/serverless-express

`body` data not passed correctly to the express app

Offen
#654 1 Kommentar 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

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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

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