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

HttpInputParameters.readTimeOut typed as string — server expects integer (ms)

Đang mở Phù hợp với người mới
#138 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
91/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
typescript
Lĩnh vực
api

Hướng nghiên cứu

Bắt đầu tại src/open-api/types.ts:156 và so sánh readTimeOut với trường connectionTimeOut được định kiểu chính xác ở dòng ngay trước đó. Thay đổi khai báo để chấp nhận các giá trị số tính bằng mili giây, sau đó xác minh rằng các kiểm tra TypeScript đều đạt và kiểu công khai khớp với trường Integer của máy chủ.

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

Mô tả

bug

Summary

HttpInputParameters.readTimeOut is typed as string in src/open-api/types.ts:156.
The server's HttpTask.Input.readTimeOut is Integer (milliseconds). The sibling field
connectionTimeOut is correctly typed as number. The inconsistency is a bug.

Server baseline

Conductor 3.32.0-rc.9

Evidence

Static:

// src/open-api/types.ts:155-156
connectionTimeOut?: number;   // ✓ correct
readTimeOut?: string;         // ✗ should be number

Server:

// http-task/.../HttpTask.java:275
private Integer readTimeOut = 3000;

Live (2026-07-16): Server accepts both readTimeOut: 3000 (integer) and
readTimeOut: "3000" (string) due to Jackson coercion. However, the TypeScript type
is still wrong — passing a number for readTimeOut is a compile-time error with the
current type definition.

Impact

TypeScript users who pass readTimeOut: 5000 (the natural form) get a type error and
must write readTimeOut: "5000" (string), which is semantically confusing and diverges
from connectionTimeOut: 5000 (number).

Proposed fix

// src/open-api/types.ts:156
readTimeOut?: number;   // was: string

Related

Discovered during systematic SDK audit against Conductor OSS 3.32.0-rc.9.

Ngôn ngữ chính
TypeScript
Star
58
Fork
20
Merge trung bình
6 ngày 5 giờ
Pull request đã merge (30 ngày)
3

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

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 conductor-oss/javascript-sdk

Tất cả issue của conductor-oss/javascript-sdk

Issue tương tự

Thêm issue về TypeScript

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.