Add extra fields in ServiceException#180
Conversation
- may enhance ServiceException with extra fields - limit values to simple objects (nor array nor collection) - add extra fields in error body - add unit tests - note: add test dependency to JSONassert, license Apache 2.0
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
Codecov Report
@@ Coverage Diff @@
## master #180 +/- ##
============================================
+ Coverage 80.06% 80.24% +0.17%
- Complexity 1715 1721 +6
============================================
Files 159 159
Lines 5718 5734 +16
Branches 754 757 +3
============================================
+ Hits 4578 4601 +23
+ Misses 856 849 -7
Partials 284 284
Continue to review full report at Codecov.
|
|
I signed it! |
|
Wrong email address in commit |
The goal of these changes is to be able to add some extra fields in ServiceException, to give more details about the error. These fields may be documented by the service and used on the client side (diagnosis, help, ...).
For example, the field 'extendedHelp' in this reply