Menu

#1 suds client throws base Exception()

open
nobody
None
5
2010-05-30
2010-05-30
JoNO
No

One example is in client.py:

if self.options.faults:
raise Exception((status, reason))

Instead it should rise transport.TransportError.

Other example:
raise Exception('(reply|fault) expected when msg=None')
One option would be to rise ValueError or a suds base exception.

Another one:
raise Exception, 'not implemented'
it should be NotImplementedError, "some text"

Either way the way exceptions are being thrown across the package should be revisited.

Discussion


Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.