Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

StacBrowser construct doesn't prefix S3 keys when pathPrefix is set

Aperta Adatta ai principianti
#313 1 commento 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
85/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
aws, typescript

Direzione di ricerca

Inizia in lib/stac-browser/index.ts, nella configurazione di BucketDeployment intorno alle righe 50-55, e analizza come viene gestito pathPrefix insieme al publicPath dell’app compilata. Verifica il comportamento del prefisso normalizzato per i valori con o senza slash iniziali e finali, quindi testa sia i casi con pathPrefix impostato sia quelli in cui non è impostato. Il lavoro è completato quando destinationKeyPrefix corrisponde a pathPrefix quando viene fornito e il comportamento esistente rimane invariato altrimenti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

When pathPrefix is set on the StacBrowser construct, the build correctly bakes the prefix into the app's publicPath but I think the files in BucketDeployment get uploaded to the root regardless of what pathPrefix is set to.

We have to rewrite the path downstream in order to properly serve anything in the bucket behind the prefix path.

so the proposed change would be something like:

        this.bucketDeployment = new s3_deployment.BucketDeployment(this, 'BucketDeployment', {
            destinationBucket: this.bucket,
            destinationKeyPrefix: props.pathPrefix?.replace(/^\/+|\/+$/g, ''),
            sources: [s3_deployment.Source.asset(buildPath)],
            memoryLimit: 1024,
            ephemeralStorageSize: Size.mebibytes(1024),
          });
Acceptance Criteria
  • destinationKeyPrefix is set to match pathPrefix when it's provided
  • The existing behavior is unchanged when pathPrefix isn't set
  • Testing done
Lingua principale
Python
Stelle
29
Fork
6
Merge medio
4h 4m
PR unite (30g)
5

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

  • Nessun Dockerfile né file Docker Compose
  • Ha un modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di developmentseed/eoapi-cdk

Tutte le issue di developmentseed/eoapi-cdk

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.