serverless-nextjs/serverless-next.js

`originUrl.includes("s3")` produces false positives

Open

#761 ouverte le 3 nov. 2020

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)TypeScript (447 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (3 998 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Describe the bug

Hi there! 👋

I'm adding an origin of this form http://foo.s3-website-eu-west-1.amazonaws.com.

As other issues revealed those kind of origins are not an S3OriginConfig but a CustomOriginConfig.

So it might help to exclude these kind of origins, or maybe allow setting an extra flag to to mark an origin as non-S3OriginConfig.

here's the problematic line of code: https://github.com/serverless-nextjs/serverless-next.js/blob/e7b5c1d302eef1daf32d150e0f12f94d4311eb0a/packages/serverless-components/aws-cloudfront/lib/getOriginConfig.js#L18

Guide contributeur

Direction de recherche
Examinez la logique dans `getOriginConfig.js` qui vérifie `originUrl.includes("s3")` et ajoutez une condition pour exclure les points de terminaison de sites Web S3 (par exemple, les URL contenant 's3 website ') d'être traités comme des origines S3.
Stack technique
typescript
Domaine
devopsinfrastructure
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitAWS CloudFront basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65