Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#313 1 bình luận 2 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
85/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
aws, typescript
Lĩnh vực
cloud, infrastructure

Hướng nghiên cứu

Bắt đầu trong lib/stac-browser/index.ts, tại cấu hình BucketDeployment quanh các dòng 50-55, và kiểm tra cách pathPrefix được xử lý cùng với publicPath của ứng dụng đã build. Xác minh behavior của prefix đã được chuẩn hóa đối với các giá trị có hoặc không có dấu gạch chéo bao quanh, sau đó kiểm thử cả các trường hợp pathPrefix được đặt và không được đặt. Hoàn tất khi destinationKeyPrefix khớp với pathPrefix khi được cung cấp và behavior hiện tại không thay đổi trong trường hợp còn lại.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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
Ngôn ngữ chính
Python
Star
29
Fork
6
Merge trung bình
9 giờ 27 phút
Pull request đã merge (30 ngày)
2

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

  • Không có Dockerfile hay tệp Docker Compose
  • Có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của developmentseed/eoapi-cdk

Tất cả issue của developmentseed/eoapi-cdk

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.