conductor-oss / conductor-oss/python-sdk

WaitTask base class sets wrong inputParameter key 'wait_until' instead of 'until'

Ouverte Adaptée aux débutants
#426 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
bug
Langage dominant
Python
Étoiles
104
Forks
42
Merge moyen
2 j 1 h
PR mergées (30 j)
3

Description

## Summary

Tested against: **Conductor OSS 3.32.0-rc.9**

`WaitTask.__init__(wait_until=...)` sets the inputParameter key `"wait_until"`,
but the Conductor server reads `"until"`. The task is accepted at registration
but never triggers at runtime — the workflow stays RUNNING indefinitely.

## Reproduction

```python
from conductor.client.workflow.task.wait_task import WaitTask

t = WaitTask("my_wait", wait_until="2024-01-01 00:00 UTC")
print(t.input_parameters)
# {'wait_until': '2024-01-01 00:00 UTC'} ← WRONG KEY
```

Start a workflow containing this task. The task status stays `IN_PROGRESS`
forever — the server never sees `"until"` so the condition is never evaluated.

## Root cause

`wait_task.py` line 26:
```python
self.input_parameters = {"wait_until": wait_until} # wrong key
```

Server constant (`Wait.java`):
```java
public static final String UNTIL_INPUT = "until";
```

## Fix

```python
self.input_parameters = {"until": wait_until}
```

Note: the `WaitUntilTask` subclass already uses the correct key `"until"` and
is unaffected. Only `WaitTask.__init__()` called directly with `wait_until=` is broken.

## Verified against

Conductor server **3.32.0-rc.9**, confirmed via live test in
[conductor-oss/conductor-test-harness](https://github.com/nthmost-orkes/conductor-test-harness):
task remained `RUNNING` after 10 s with `wait_until` key; `WaitUntilTask` with
`until` key completed immediately.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Start with wait_task.py at line 26 and compare WaitTask.__init__() with the unaffected WaitUntilTask subclass and the server’s Wait.java constant. Done means direct WaitTask use emits the server-recognized input parameter and no longer leaves the task running indefinitely; verify the serialized parameters and workflow behavior described in the reproduction.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
backend
Type d'issue
Bug
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
Calme
Clarté
Clairement spécifiée
Accessibilité débutants
82/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.