Skip to content

Fix default message check#135

Merged
afgane merged 1 commit intogalaxyproject:masterfrom
mvdbeek:fix_exception_check_again
Jun 4, 2019
Merged

Fix default message check#135
afgane merged 1 commit intogalaxyproject:masterfrom
mvdbeek:fix_exception_check_again

Conversation

@mvdbeek
Copy link
Copy Markdown
Member

@mvdbeek mvdbeek commented Jun 4, 2019

requests.exceptions.ConnectionError Exceptions don't have a body
attribute, but bioblend's ConnectionError __str__ includes
the body:

    def __str__(self):
        return "{0}: {1}".format(self.args[0], self.body)

requests.exceptions.ConnectionError Exceptions don't have a body
attribute, but bioblend's ConnectionError `__str__` includes
the body:
```
    def __str__(self):
        return "{0}: {1}".format(self.args[0], self.body)
```
Copy link
Copy Markdown
Contributor

@afgane afgane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@afgane afgane merged commit 77b5bd8 into galaxyproject:master Jun 4, 2019
@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 4, 2019

Thanks Enis!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants