conductor-oss / conductor-oss/javascript-sdk

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

Aperta Adatta ai principianti
#138 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug
Lingua principale
TypeScript
Stelle
58
Fork
20
Merge medio
1g 13h
PR unite (30g)
7

Descrizione

## 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.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Start at src/open-api/types.ts:156 and compare readTimeOut with the correctly typed connectionTimeOut field at the preceding line. Change the declaration so numeric millisecond values are accepted, then verify the TypeScript checks pass and that the public type matches the server's Integer field.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
typescript
Ambito
api
Tipo di issue
Bug
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Tranquilla
Chiarezza
Specificata chiaramente
Idoneità per principianti
91/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.