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

BUG: HTTPTarget sends conflicting Content-Length and Transfer-Encoding headers

Đã đóng
  • Có gì không ổn?
#3,202 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 2 ngày

@Kaap10 đang làm issue này rồi.

Từ ngày 6/10/2026.

  • #3008 của @Kaap10 — đang mở

Đá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
35/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
api

Hướng nghiên cứu

Start in parse_raw_http_request in pyrit/prompt_target/http_target/http_target.py, where only content-length is popped from the parsed headers. Check open PR #3008 first, since it already touches this branch and may overlap. Done means neither framing header survives parsing and the body is sent intact, which the suggested MockTransport test in tests/unit/prompt_target/target/test_request_tracing.py can pin.

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

Mô tả

Bug: triage
Describe the bug

HTTPTarget.parse_raw_http_request strips content-length from the parsed header dict so that httpx can compute the correct value for the (possibly converter-modified) body:

https://github.com/microsoft/PyRIT/blob/main/pyrit/prompt_target/http_target/http_target.py#L290-L291

if "content-length" in headers_dict:
    del headers_dict["content-length"]

transfer-encoding is not stripped. When a raw request contains Transfer-Encoding: chunked (common when pasting from Burp), that header survives into the request, httpx/h11 chunk-frames the body, and httpx still adds its own Content-Length. Both framing headers ship on the same request.

RFC 9112 §6.1 states a sender MUST NOT send Content-Length in any message that contains a Transfer-Encoding header field. §6.3 requires recipients that receive both to treat the message as unrecoverable, and recommends intermediaries forwarding such a message strip Content-Length.

Two concrete impacts:

  1. Silent truncation / mojibake. The chunk-framed body is 20 bytes on the wire but Content-Length advertises 11 (the decoded length). Any intermediary or server that honors Content-Length reads a prefix of the chunked framing. With multibyte content this truncates mid-UTF-8 and the operator sees corrupted text rather than an error.
  2. Request smuggling shape. This is textbook CL.TE desync. PyRIT emits it with no warning, so an operator cannot tell whether an anomalous target response is a finding or a PyRIT framing artifact. For a tool whose output is security evidence, that ambiguity is the real cost.

This is pre-existing on main and unrelated to any open PR; I confirmed it reproduces both on main and on the branch in #3008.

Steps/Code to Reproduce
import asyncio, socket, threading

from pyrit.executor.attack import PromptSendingAttack
from pyrit.prompt_target import HTTPTarget
from pyrit.setup import IN_MEMORY, initialize_pyrit_async

captured = []
srv = socket.socket(); srv.bind(("127.0.0.1", 0)); srv.listen(1)
port = srv.getsockname()[1]


def serve():
    conn, _ = srv.accept()
    buf = b""
    while b"0\r\n\r\n" not in buf:
        chunk = conn.recv(65535)
        if not chunk:
            break
        buf += chunk
    captured.append(buf)
    conn.sendall(b"HTTP/1.1 200 OK\r\nContent-Length: 2\r\n\r\nok")
    conn.close()


threading.Thread(target=serve, daemon=True).start()

raw = (
    "POST / HTTP/1.1\r\n"
    f"Host: 127.0.0.1:{port}\r\n"
    "Transfer-Encoding: chunked\r\n"
    "Content-Length: 11\r\n"
    "\r\n"
    "{PROMPT}"
)


async def main():
    await initialize_pyrit_async(memory_db_type=IN_MEMORY)
    target = HTTPTarget(http_request=raw, prompt_regex_string="{PROMPT}", use_tls=False)
    await PromptSendingAttack(objective_target=target, max_attempts_on_failure=0).execute_async(
        objective="hello world"
    )
    print(captured[0].decode())


asyncio.run(main())
Expected Results

Exactly one framing header on the wire. Either transfer-encoding is stripped alongside content-length so httpx owns framing, or chunked framing is honored and no Content-Length is emitted.

Actual Results

Both framing headers are sent, and the Content-Length value does not describe the bytes actually transmitted:

POST / HTTP/1.1
host: 127.0.0.1:49678
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
User-Agent: python-httpx/0.28.1
transfer-encoding: chunked
content-length: 11

b
hello world
0

Body is chunk-framed (b\r\nhello world\r\n0\r\n\r\n, 20 bytes) while content-length: 11 describes the decoded length. No warning or error is raised.

Suggested fix

Strip both hop-by-hop framing headers in parse_raw_http_request and let httpx own framing, consistent with the existing intent for content-length:

for framing_header in ("content-length", "transfer-encoding"):
    headers_dict.pop(framing_header, None)

A regression test using httpx.MockTransport (matching the pattern in tests/unit/prompt_target/target/test_request_tracing.py) can pin that neither header survives parsing and that the body is sent intact.

If preserving operator-supplied chunked framing is considered in scope, that is a larger change and should be explicit rather than the current accidental middle state.

Screenshots

N/A

Versions
  • OS: Windows
  • Python version: 3.14
  • PyRIT version: installed from main in editable mode (1.2.0.dev0)
  • httpx: 0.28.1
Ngôn ngữ chính
Python
Star
4.6k
Fork
944
Merge trung bình
2 ngày 16 giờ
Pull request đã merge (30 ngày)
350

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 microsoft/PyRIT

Tất cả issue của microsoft/PyRIT

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.