Docs: socket.htons() and friends raise ValueError for negative input since 3.14
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
- Loại issue
- Tài liệu
- Độ 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
- documentation
Hướng nghiên cứu
Đọc Doc/library/socket.rst và xem tài liệu về htons(), ntohs(), htonl() và ntohl(). Xác nhận thay đổi ngoại lệ được mô tả dựa trên ví dụ hoặc các bài kiểm thử được cung cấp, sau đó ghi lại hành vi trong phiên bản 3.14 cho cả bốn hàm hoặc trong ghi chú chuyển đổi sang phiên bản 3.14. Công việc hoàn tất khi tài liệu phân biệt ValueError đối với đầu vào âm với OverflowError đối với giá trị quá lớn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Documentation
Doc/library/socket.rst, htons() (and ntohs()):
.. versionchanged:: 3.10
Raises OverflowError if x does not fit in a 16-bit unsigned integer.
Since 3.14 a negative argument raises ValueError ("Cannot convert negative
int"); only too-large values still raise OverflowError. htonl()/ntohl() behave the same way:
import socket
for f in (socket.htons, socket.ntohs, socket.htonl, socket.ntohl):
for v in (-1, 1 << 40):
try:
f(v)
except Exception as e:
print(f.__name__, v, type(e).__name__)
| Version | -1 |
1 << 40 |
|---|---|---|
| 3.13 | OverflowError | OverflowError |
| 3.14 / 3.15 | ValueError | OverflowError |
This probably came with gh-132987 (__index__() support in the socket module).
It may be intended, but it is not documented, and except OverflowError
code written against the 3.10 note no longer catches negative input.
Suggested fix: add a versionchanged:: 3.14 note to all four functions, or
mention it in What's New in 3.14 (Porting).
Linked PRs
- gh-159047
- Ngôn ngữ chính
- Python
- Star
- 77.6k
- Fork
- 37.6k
- Merge trung bình
- 1 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 608
Chuẩn bị môi trường
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
- Đọc hướng dẫn đóng góp
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 python/cpython
-
Docs: problem with page "msvcrt — Useful routines from the MS VC runtime"Có thể đã có người làm @jikekei đã nhận 1 ngày trước. Đang mởdocs
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
python/cpython#159072 · 6 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
docs
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
python/cpython#159061 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
`http.client.HTTPSConnection._tunnel` raises `AttributeError: 'NoneType' object has no attribute 'encode'` when header value is `None`Có thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở3.14 3.15 3.16 stdlib type-bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
python/cpython#159044 · 17 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
A minor problem with the section "6.10.2. Membership test operations" from the language reference.Đang mởdocs pending
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
python/cpython#159034 · 5 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
stdlib topic-profiling type-bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của python/cpython
Issue tương tự
-
Broken links found in docsĐang mởdocs pydanty:is-working
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
pydantic/pydantic-ai#10113 ·
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 77/100
GreedyBear-Project/GreedyBear#1727 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
ARM-software/CMSIS-DSP#354 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
kiat/codeAssist#391 ·
-
needs triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
darnitdevorg/darnit#587 ·
Maintainer thường phản hồi trong vòng 1 ngày