CodeGenieApp / CodeGenieApp/serverless-express

Req.url incorrectly evaluated for paths with nested proxy - ApiGateway

Offen
#400 5 Kommentare 7 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

GIven the following resource `/testing/{proxy+}` a call to `/testing/creategroup` is evaluated to `/creategroup` rather than `/testing/creatgroup`.

Looking at [getPathWithQueryStringParams](https://github.com/vendia/serverless-express/blob/671f914429e9708bfb1ee6ba76b5947a917592df/src/event-sources/utils.js#L7), it appears that if proxy pathParameter is present, it is simply used, discarding the other parts. Would substituting the value for proxy for the token {proxxy+} into `event.resource` (/testing/{proxy+}) be a potential approach to fixing this?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in src/event-sources/utils.js bei getPathWithQueryStringParams und verwende die gemeldete Ressource /testing/{proxy+} sowie den Anfragepfad /testing/creategroup als Reproduktionsfall. Verfolge, wie der Proxy-Pfadparameter mit event.resource kombiniert wird, und überprüfe anschließend, dass die resultierende Req.url bei verschachtelten Proxy-Pfaden das Präfix /testing beibehält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
aws, express, javascript
Bereich
api, backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

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