bolt-python icon indicating copy to clipboard operation
bolt-python copied to clipboard

Make Exeption traceable in lazy listener

Open jimmymaise opened this issue 3 years ago • 6 comments

(Describe your issue and goal here) Currently, using lazy listener for FAAS (such as AWS Lambda), it's really difficult to debug. If we have a exception, we just have something like

Failed to run an internal function (Exception ABC)

We don't know exactly, what is the error, traceback, error line etc...

 try:
            await internal_func(
                **build_async_required_kwargs(
                    logger=logger,
                    required_arg_names=arg_names,
                    request=request,
                    response=None,
                    this_func=internal_func,
                )
            )
        except Exception as e:
            logger.error(f"Failed to run an internal function ({e})")

So should we have an option to enable the exception traceback or provide some meaningful error information to debug instead of just simply message like that

On the other hand, the document about error handling does not work with lazy listener so we also need to update. https://slack.dev/bolt-python/concepts#errors

Category (place an x in each of the [ ])

  • [ ] slack_bolt.App and/or its core components
  • [ ] slack_bolt.async_app.AsyncApp and/or its core components
  • [ ] Adapters in slack_bolt.adapter
  • [x ] Others

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

jimmymaise avatar Apr 20 '22 06:04 jimmymaise

Hi @jimmymaise, thanks for the feedback. We will look into this. This can be improved in future versions and/or needs to be clearly mentioned in the documents.

seratch avatar Apr 20 '22 13:04 seratch

We are facing the same issue as @jimmymaise, I think it would be helpful to have the exception traceback (we are using sentry and would love to see what line is causing the issue).

ddhanak avatar Aug 18 '22 15:08 ddhanak

We are also facing this issue; it's often unclear what is causing an exception (or swallowing them entirely).

kevgliss avatar Jan 10 '23 18:01 kevgliss

I'm running into this issue frequently as well. I'd love to see this updated

angrychimp avatar Sep 11 '23 16:09 angrychimp