`cf workers secrets bulk` silently no-ops on the plain body shape; `secrets update` accepts invalid names

Đang mở
  • Có gì không ổn?
#102 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

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

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
52/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
api, cli, security

Hướng nghiên cứu

Start by reproducing the documented cf workers secrets bulk and secrets update commands, then trace the workers secrets command, help, and schema paths. Compare plain and wrapped bodies, .env input, and invalid names; done means the behavior is explicit, the body shape is documented in help and schema, and invalid names are rejected locally.

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

Mô tả

upstream

cf version: 1.0.0-beta.5, macOS

1. cf workers secrets bulk silently does nothing with the plain (RFC 7396 merge-patch) body.

cf workers secrets bulk --worker W --body '{"API_KEY":{"type":"secret_text","text":"v"}}'

This returns 200 and exits 0, and the response lists the bindings, but the secret's value doesn't change (checked for 20 s). Only the wrapped shape {"secrets":{"API_KEY":{...}}} applies it.

  • --file with the plain shape: the same silent no-op.
  • --file with a .env file: [10026] Could not parse request body.
  • --help doesn't show the body shape, and cf schema workers secrets bulk shows requestBodyFields: [].

2. cf workers secrets update accepts invalid names. cf workers secrets update "bad-name!" --worker W --type secret_text --text v creates the secret (exit 0), although the help says the name must be "A JavaScript variable name". It can only be removed with delete "bad-name!" -f.

Expected: reject a body that won't apply, or accept the plain shape; document the body in --help and the schema; validate secret names locally.

Related, but a different problem: #96 (secrets update replaces the whole set).

Found while evaluating cf for a dev workflow: https://github.com/joeblew999/cf-cli-test (FINDINGS.md, findings/).

Ngôn ngữ chính
TypeScript
Star
993
Fork
65
Merge trung bình
21 giờ 40 phút
Pull request đã merge (30 ngày)
211

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

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 cloudflare/cf

Tất cả issue của cloudflare/cf

Issue tương tự

Thêm issue về TypeScript

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.