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

Generated code reuses the same err variable

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,924 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
70/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
go
Lĩnh vực
api, backend

Hướng nghiên cứu

Xem xét trình tạo mã tạo ra các trình xử lý API, có thể trong một tệp như pkg/codegen/templates.go hoặc tương tự. Tìm nơi runtime.BindQueryParameter được gọi và biến lỗi được tái sử dụng. Thay đổi mẫu để sử dụng if err := ... nhằm tránh shadowing. Chạy các bài kiểm tra của trình tạo để đảm bảo đầu ra là chính xác.

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

Mô tả

Generated API handlers reuse the same err variable when binding params like

err = runtime.BindQueryParameter(...)
if err != nil {...}

which could lead to potential security issue on race condition. e.g. If the error was a global variable, when a thread fails the bind and returns an error while the other passes the bind without error, the error could get nullified in the error case. But since we are not using a global error variable in the generated code, this should not really be an issue here.

But anyway it could be good if we can change the generated code to declare new error variable instead of shadowing an existing one.

if err := runtime.BindQueryParamter(...);  err != nil {...}
Ngôn ngữ chính
Go
Star
8.6k
Fork
1.1k
Merge trung bình
14 giờ 45 phút
Pull request đã merge (30 ngày)
29

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 oapi-codegen/oapi-codegen

Tất cả issue của oapi-codegen/oapi-codegen

Issue tương tự

Thêm issue về Go

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.