scripts/verify-pack: exits 0 when no pack directory is given
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- bash
- Lĩnh vực
- cli
Hướng nghiên cứu
Read scripts/verify-pack around lines 401–405 and the usage() function at line 48. Check how the no-argument path handles output and exit status; done when it reports the error on stderr and exits nonzero. Pull request #141 is already open for this issue.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
if [[ ${#TARGETS[@]} -eq 0 ]]; then
echo "Error: No pack directory specified."
echo ""
usage
fi
usage() ends with exit 0 (L48).
Problem: bash scripts/verify-pack with no argument prints an error but returns exit code 0. A script that calls verify-pack "$PACK" with an empty $PACK thinks that the check passed. The error also goes to stdout, not stderr.
Fix: At line 404, use usage >&2; exit 1 (or give usage() an exit-code argument).
Found by a code read at commit 1e50295. The code was read, but not run on a real instance.
- Ngôn ngữ chính
- Shell
- Star
- 73
- Fork
- 11
- Merge trung bình
- 6 giờ 46 phút
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của inceptionstack/lowkey
-
bug good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
inceptionstack/lowkey#135 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
inceptionstack/lowkey#134 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 94/100
inceptionstack/lowkey#133 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
inceptionstack/lowkey#131 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
inceptionstack/lowkey#130 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của inceptionstack/lowkey
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
sclorg/s2i-ruby-container#660 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
wazuh/wazuh-docker#2732 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Route 53 DNS analytics/insights adds an observability signal for the DNS layerCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởdocumentation enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100