Reject ineffective fingerprint-forwarding carrier configurations

Đang mở
#30 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 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
48/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
rust
Lĩnh vực
networking

Hướng nghiên cứu

Theo dõi việc xây dựng route tĩnh và các route được cung cấp thông qua resolveConnection, bắt đầu từ cảnh báo stderr hiện có về việc chuyển tiếp dấu vân tay không có hiệu lực. Xem xét các bài kiểm thử liên quan đến route và bổ sung phạm vi bao phủ cho các tổ hợp route tĩnh và route đã được resolve hợp lệ và không hợp lệ; được xem là hoàn tất khi việc xây dựng hoặc resolve từ chối các carrier không tương thích kèm theo khuyến nghị dành riêng cho route.

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

Mô tả

Follow-up from the review of #23.

Problem

A route can request forwardFingerprint: ja3 or ja4 without selecting a carrier that can actually deliver it.

The HTTP-header carrier requires terminated plaintext HTTP/1. Passthrough and HTTP/2 require sourceAddressHeader: proxyProtocolV2. Static routes currently emit only an stderr warning for ineffective combinations, allowing an apparently healthy deployment that silently drops the requested fingerprint. Routes supplied through resolveConnection do not receive equivalent warning or validation.

This leaves downstream controls believing they receive an authoritative fingerprint when no signal is delivered.

Acceptance criteria

  • Reject configurations where fingerprint forwarding has no viable carrier, rather than relying on stderr.
  • Apply identical validation to static routes and routes supplied through resolveConnection.
  • HTTP-header forwarding must require TLS termination and an HTTP/1 upstream path.
  • Passthrough and HTTP/2 forwarding must require PROXY protocol v2.
  • Validate incompatible UDS/Harper combinations where the immediate consumer does not support the selected PROXY protocol version.
  • Return a construction/resolution error that identifies the route and recommends the compatible carrier.
  • Add tests for valid and invalid static and resolved-route combinations.
Ngôn ngữ chính
Rust
Star
1
Fork
0
Merge trung bình
19 giờ 12 phút
Pull request đã merge (30 ngày)
13

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 HarperFast/symphony

Tất cả issue của HarperFast/symphony

Issue tương tự

Thêm issue về Rust

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.