conductor-oss / conductor-oss/javascript-sdk

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

Offen Anfängerfreundlich
#138 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug
Vorherrschende Sprache
TypeScript
Sterne
58
Forks
20
Ø Merge
1 T. 13 Std.
Gemergte PRs (30 T.)
7

Beschreibung

## 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:**
```typescript
// src/open-api/types.ts:155-156
connectionTimeOut?: number; // ✓ correct
readTimeOut?: string; // ✗ should be number
```

**Server:**
```java
// 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

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

## Related

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne bei src/open-api/types.ts:156 und vergleiche readTimeOut mit dem korrekt typisierten Feld connectionTimeOut in der vorherigen Zeile. Ändere die Deklaration so, dass numerische Millisekundenwerte akzeptiert werden, und überprüfe anschließend, dass die TypeScript-Prüfungen erfolgreich sind und der öffentliche Typ mit dem Integer-Feld des Servers übereinstimmt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
api
Issue-Typ
Bug
Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Aktivitätsstatus
Ruhig
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
91/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.