You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Error handling is not intuitive for the Slack SDK. The result object returned by an API call has an ok property, which indicates the API will never reject/throw and you need to check the result to determine the success. However, in practice the SDK will reject/throw. We are not currently handling this in the code. This is particularly bad in the stream lambda since we could stuck trying to process the same record forever when we hit a Slack API error that is permanent.
The text was updated successfully, but these errors were encountered:
I would agree with this issue:
slackapi/node-slack-sdk#775
Error handling is not intuitive for the Slack SDK. The result object returned by an API call has an
ok
property, which indicates the API will never reject/throw and you need to check the result to determine the success. However, in practice the SDK will reject/throw. We are not currently handling this in the code. This is particularly bad in the stream lambda since we could stuck trying to process the same record forever when we hit a Slack API error that is permanent.The text was updated successfully, but these errors were encountered: