crossbario / crossbario/autobahn-python

ApplicationError Exception Formatting

Open
#784 4 comments 0 reactions 1 assignee View on GitHub

@meejah is already working on this.

Since Mar 12, 2021.

bug
Dominant language
Python
Stars
2.5k
Forks
768
PR merge metrics
No merged PRs in 30d

Description

If I throw an exception in a called function the other host prints the unicode version of ApplicationError, which includes the traceback thrown by the remote function interleaved with newlines. If appears the unicode function on ApplicationError expects the traceback to be a list of lines, but it is receiving a string. (As seen here: https://github.com/crossbario/autobahn-python/blob/master/autobahn/wamp/exception.py#L241) As far as I can tell this originates from https://github.com/twisted/twisted/blob/68d2afa5c2776c242d6bb942813b293b027b837c/src/twisted/python/failure.py#L541

Is there a place I should be overriding this behavior, or is this unintentional? I could really use a guide on error handling with autobahn in general.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.