CodeGenieApp / CodeGenieApp/serverless-express

requests to /foo/bar to an API Gateway (v1) resource of /foo/{proxy+} return 404 as they are routed to /bar

Ouverte
#648 9 commentaires 4 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
5.3k
Forks
674
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

I'm having the same issue as in this comment https://github.com/vendia/serverless-express/issues/454#issuecomment-1322898350 (should be probably reopened?)

Upgraded from the legacy `aws-serverless-express` and for some reason it takes the path from `pathParameters.proxy` (which is `bar`) and not the request.path (which is `/foo/bar`)

I'll be happy to work on a PR if you indeed confirm it's a bug

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par reproduire une requête API Gateway v1 vers la ressource /foo/{proxy+} et comparez pathParameters.proxy avec request.path. Suivez la manière dont le chemin de la requête est sélectionné après la mise à niveau depuis aws-serverless-express, en vous appuyant sur le commentaire lié de l’issue. Le travail est considéré comme terminé lorsque /foo/bar atteint l’application Express sous la forme /foo/bar plutôt que /bar, avec un test de régression couvrant ce comportement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
aws, express, javascript, node.js
Domaine
api, backend, cloud
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.