Add more detail for API errors.

Review Request #4396 — Created Aug. 5, 2013 and submitted

Information

RBTools
master

Reviewers

Add more detail for API errors.

We now log debug output for any API errors. The output shows the payload
of the error, making it easier to see what went wrong.

In the case of a failed diff upload, we also include the wrapped error
class's text in the message, instead of the raw error string we get from
the server.
Tested locally with a bad payload. Saw useful error output in the displayed
string and in the debug output.
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/commands/post.py
        rbtools/api/request.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/commands/post.py
        rbtools/api/request.py
      Ignored Files:
    
    
  2. 
      
SM
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...