CodeGenieApp / CodeGenieApp/serverless-express

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

オープン
#400 コメント 5 件 リアクション 7 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
5.3k
フォーク
676
PR マージ指標
30日以内にマージされた PR はありません

説明

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?

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

src/event-sources/utils.js の getPathWithQueryStringParams から始め、報告されたリソース /testing/{proxy+} とリクエストパス /testing/creategroup を再現ケースとして使用します。プロキシパスパラメータが event.resource とどのように結合されるかを追跡し、そのうえで、結果として得られる Req.url がネストされたプロキシパスでも /testing プレフィックスを保持することを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
aws, express, javascript
領域
api, backend
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。