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

ssl_multicert: entries without dest_ip can override the default certificate

Đang mở
  • Có gì không ổn?
#13,798 0 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 2 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
48/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ệ
cpp
Lĩnh vực
networking

Hướng nghiên cứu

Start by tracing how ssl_multicert entries are parsed and how the configuration converter handles omitted dest_ip; the issue does not name specific files or tests. Check the relevant tests for YAML and legacy ssl_multicert.config inputs, then run them. Done means omitted dest_ip stays empty, converted entries no longer gain "*", and the explicitly configured default certificate is used for clients without a matching SNI.

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

Mô tả

On master, an entry without dest_ip is treated as dest_ip: "*". It competes with the explicitly configured default certificate.

For example:

ssl_multicert:
  - ssl_cert_name: named.pem
    ssl_key_name: named.key

  - dest_ip: "*"
    ssl_cert_name: default.pem
    ssl_key_name: default.key

Expected: Clients with no SNI or an unmatched SNI receive default.pem.

Actual: The first entry to register "*" wins. Startup loads certificates concurrently, so the default can change between restarts. With the default reload concurrency of one, the first entry wins, making named.pem the default after a reload.

This affects both YAML and legacy ssl_multicert.config files on master. It does not affect 10.2.x and should be fixed before 11.0.

Workaround: Set dest_ip: "" on entries that should not be the default.

The fix should preserve an omitted dest_ip as empty. The configuration converter also needs updating: it currently writes dest_ip: "*" into entries that omitted it, so previously converted files will need correction.

Ngôn ngữ chính
C++
Star
2k
Fork
878
Merge trung bình
3 ngày 15 giờ
Pull request đã merge (30 ngày)
94

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 apache/trafficserver

Tất cả issue của apache/trafficserver

Issue tương tự

Thêm issue về C++

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.