Why is Github::Error::GithubError#message a Hash and not a String? about github HOT 5 CLOSED

Hi Darren, yeah this looks too brittle for what it needs to be. I will try to tackle this one and simplify it asap, however in the meantime if you have any suggestions how this should be handled please send in a patch. Thanks Piotr

from github.

Thank you for your feedback. I may send you a patch when I have time :)

from github.

Hi Darren, I've spent some time on this issue and rewrote the error handling to always return a string message. All the response decoding and message extraction now happens inside the gem, therefore your code can be simplified as follows:

begin
    #Do something with github_api gem
rescue Github::Error::GithubError => e
  puts e.message => #e.g. 'key is already in use'
end

Please note that you can further refine your error catching to only GitHub service errors by using Github::Error::ServiceError etc.. plus I've updated the README with error handling information for future reference.

I hope this helps. Please download the v0.6.4 gem release and let us know if that's working for you?

P

from github.

I tried it out and it works! The message content includes the HTTP verb and the url (as well as the error message) by the way, for example, I received: "POST https://api.github.com/repos/jensendarren/try_git/keys?access_token=003betc7uwehblahblah7wj: 422 key is already in use"

Anyway, e.message.class is now a String which is great! Thanks Peter!

from github.

Awesome, I figured that having the explicit request information together with error message will make it more convenient for future debugging. Glad it works for you.

from github.