serverless-nextjs/serverless-next.js

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

Open

#761 aberto em 3 de nov. de 2020

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)TypeScript (447 forks)batch import
buggood first issue

Métricas do repositório

Stars
 (3.998 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Veja a linha referenciada em packages/serverless components/aws cloudfront/lib/getOriginConfig.js linha 18. A condição `originUrl.includes('s3')` identifica erroneamente endpoints de site do S3 como origens S3. Pesquise a diferença entre S3OriginConfig e CustomOriginConfig no AWS CloudFront e encontre uma maneira mais robusta de distingui los, como verificar padrões de domínio ou usar a detecção de tipo de origem do AWS SDK.
Pilha de tecnologia
typescriptnodejsaws
Domain
cloudinfrastructuredevops
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
basic TypeScriptAWS CloudFront origin config concepts
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85