REST API should log all errors that occur in downstream code
Reported by Ben Hood | 2011-05-20 16:39:31 UTC
Currently if a call to REST API blows up, the error will be propagated straight out as a 500 error with a single entry in the diffa log - hence the REST resource should catch all errors and log them appropriately on the server side. One potential way to achieve this is to use the exception mapping facility in Jersey. Also, there is a hidden bug in the DifferencesResource that catches a NoSuchElementException, which arises from an underlying session not being available.
Comments and changes to this ticket
-
Ben Hood 2011-05-31 14:08:20 UTC
- State changed from “new” to “open”
- Assigned user set to “Ben Hood”
-
Ben Hood 2011-05-31 14:09:36 UTC
Added a catch all handler for all app specific exceptions and an extra matcher for all HTTP errors in order to pass the latter through the normal processing route.
-
Ben Hood 2011-05-31 14:09:54 UTC
- State changed from “open” to “resolved”
- Assigned user changed from “Ben Hood” to “Paul Jones”
-
Ben Hood 2011-05-31 22:51:40 UTC
- State changed from “resolved” to “open”
- Assigned user changed from “Paul Jones” to “Ben Hood”
Re-opening, because I think the AbstractRestResource can probably get binned now.
-
Ben Hood 2011-05-31 22:52:53 UTC
(from [0da02ab03c83e3b7f74eb8f1a1cb60577b5c14c1]) [#217] Added MissingObject mapping, should probably further cut down on the underlying REST resource though https://github.com/lshift/diffa/commit/0da02ab03c83e3b7f74eb8f1a1cb...
-
Ben Hood 2011-06-01 09:48:19 UTC
(from [775e5183803318ec4eaf37a0605a593ef7310637]) [#217] Turned AbstractrRestResource into helper object https://github.com/lshift/diffa/commit/775e5183803318ec4eaf37a0605a...
-
Ben Hood 2011-06-01 09:49:27 UTC
- State changed from “open” to “resolved”
- Assigned user changed from “Ben Hood” to “Paul Jones”
Refactored AbstractrRestResource, re-resolving
-
Ben Hood 2011-06-01 12:58:44 UTC
- Assigned user changed from “Paul Jones” to “Ben Hood”
Re-opening since this pacth masks the NoSuchElementException thrown by the session manager
-
Ben Hood 2011-06-01 14:53:07 UTC
(from [ac5349839de220da087a2ceca67d284948eec86e]) [#217] Implemented tests to verify the 404 handling in Jersey https://github.com/lshift/diffa/commit/ac5349839de220da087a2ceca67d...
-
Ben Hood 2011-06-01 14:56:11 UTC
- Assigned user changed from “Ben Hood” to “Paul Jones”
Added 404 tests -> re-resolving
-
-
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
A Real Time Differencing Tool.
People watching this ticket
Ben Hood
Referenced by
-
217
REST API should log all errors that occur in downstream code
(from [0da02ab03c83e3b7f74eb8f1a1cb60577b5c14c1])
[#217] ...
-
217
REST API should log all errors that occur in downstream code
(from [775e5183803318ec4eaf37a0605a593ef7310637])
[#217] ...
-
217
REST API should log all errors that occur in downstream code
(from [ac5349839de220da087a2ceca67d284948eec86e])
[#217] ...
-
217
REST API should log all errors that occur in downstream code
(from [ae8d00cd9c0009cb16a86dd05e692fe8f535747c])
Merge [...