Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

retryAfter description does not match the available type enumerations #220

Open
jeremyfiel opened this issue Jul 30, 2024 · 3 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@jeremyfiel
Copy link

jeremyfiel commented Jul 30, 2024

in the description of retryAfter indicates goto or function are acceptable failure action type enumerations.

function is not defined in the type keyword description

<a name="failureRetryAfter"></a> retryAfter | `number` | A non-negative decimal indicating the seconds to delay after the step failure before another attempt SHALL be made. **Note:** if an HTTP [Retry-After](https://tools.ietf.org/html/rfc9110.html#name-retry-after) response header was returned to a step from a targeted operation, then it SHOULD overrule this particular field value. This field only applies when the `type` field value is `"retry"` or `"function"`.

A non-negative decimal indicating the seconds to delay after the step failure before another attempt SHALL be made. Note: if an HTTP Retry-After response header was returned to a step from a targeted operation, then it SHOULD overrule this particular field value. This field only applies when the type field value is "retry" or "function".


<a name="failureActionType"></a> type | `string` | **REQUIRED**. The type of action to take. Possible values are `"end"`, `"retry"`, or `"goto"`.

REQUIRED. The type of action to take. Possible values are "end", "retry", or "goto".

@frankkilcommins
Copy link
Collaborator

Nice spot. Thanks

@TristanSpeakEasy
Copy link

Have just run into this myself, does function need to be removed here or will it be added to the available types?

@jeremyfiel
Copy link
Author

Fixed here. v1.0.0-dev is not merged to main yet

9e0095f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants