Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Adding request context to exception message #54

Merged
merged 1 commit into from Feb 3, 2013

Conversation

Projects
None yet
3 participants
Contributor

jeremyf commented Jan 28, 2013

Prior to this patch, I would receive the following exception:

Errno::ECONNREFUSED (Connection refused - connect(2))

With this patch, I receive the equivalent of:

Errno::ECONNREFUSED: Connection refused - {
  :uri=>#<URI::HTTP:0x007fd5093b22a8 URL:http://localhost/some_uri>,
  :method=>:get,
  :open_timeout=>42
}
``
Adding request context to exception message
Prior to this patch, I would receive the following exception:
```
Errno::ECONNREFUSED (Connection refused - connect(2))
```

With this patch, I receive the equivalent of:
```
Errno::ECONNREFUSED: Connection refused - {
  :uri=>#<URI::HTTP:0x007fd5093b22a8 URL:http://localhost/some_uri>,
  :method=>:get,
  :open_timeout=>42
}
``

mwmitchell added a commit that referenced this pull request Feb 3, 2013

Merge pull request #54 from jeremyf/adding-connection-information-to-…
…error-handling

Adding request context to exception message

@mwmitchell mwmitchell merged commit 4d10634 into rsolr:master Feb 3, 2013

jcoyne commented on ab3b1f9 Feb 18, 2013

This is very useful. Thanks Jeremy!

solenko added a commit to solenko/rsolr that referenced this pull request Sep 11, 2013

solenko added a commit to solenko/rsolr that referenced this pull request Sep 12, 2013

solenko added a commit to solenko/rsolr that referenced this pull request Sep 12, 2013

cbeer added a commit that referenced this pull request Sep 29, 2016

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