serverless-nextjs/serverless-next.js

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

Open

#761 geöffnet am 3. Nov. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (3.998 Stars) (447 Forks)batch import
buggood first issue

Beschreibung

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

Contributor Guide

Tech Stack
typescriptnodejsaws
Domain
cloudinfrastructuredevops
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic TypeScriptAWS CloudFront origin config concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
Look at the referenced line in packages/serverless components/aws cloudfront/lib/getOriginConfig.js line 18. The condition `originUrl.includes('s3')` falsely identifies S3 website endpoints as S3 origins. Research the difference between S3OriginConfig and CustomOriginConfig in AWS CloudFront, and find a more robust way to distinguish them, such as checking domain patterns or using the AWS SDK's origin type detection.