Multi-value tornado headers are fetched with `get()` rather than `get_list()`

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,109 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.

  • #1110 của @Prognosticate-X — đã đóng, không merge
  • #1113 của @Fire162 — đã đóng, không merge
  • #1114 của @X6M-Work — đã đóng, không merge

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

Hướng nghiên cứu

Bắt đầu từ bộ phân tích tornado, nơi các lần tra cứu header multidict gọi get(...) trên đối tượng HTTPHeaders của tornado. Tái hiện lỗi bằng hai lệnh gọi add trên cùng một tên header, sau đó so sánh get với get_list. Hoàn thành khi một bài kiểm thử hồi quy cho thấy các header lặp lại được trả về dưới dạng danh sách và hành vi của bộ phân tích đối với chúng đã được sửa. Hãy đọc chính sách đóng góp có sử dụng AI của dự án trước, vì lần thử trước đã bị đóng theo chính sách đó.

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

Mô tả

In the tornado parser, we have multidict support for headers which gets data using get(...). This combines headers, and therefore gives wrong results.
Given the setup:

>>> import tornado
>>> h = tornado.httputil.HTTPHeaders()
>>> h.add("a", "foo")
>>> h.add("a", "bar")

Compare:

>>> h.get("a")
'foo,bar'
>>> h.get_list("a")
['foo', 'bar']

We should add a regression test for this and a fix.

This was flagged in #1108, which was summarily closed as a low-quality PR in violation of our AI policy.

Ngôn ngữ chính
Python
Star
1.4k
Fork
171
Merge trung bình
8 giờ 47 phút
Pull request đã merge (30 ngày)
12

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 marshmallow-code/webargs

Tất cả issue của marshmallow-code/webargs

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.