feat:sentinel can feel exception though application has configured ExceptionHandler
Describe what this PR does / why we need it
sentinel can feel exception though application has configured ExceptionHandler
Does this pull request fix one issue?
#3141
Describe how you did it
Add a class SentinelExceptionAware which implement HandlerExceptionResolver and put exception into request.
Describe how to verify it
UT & add it in my project
Special notes for reviews
Thanks for your work.
Contributor License Agreement was signed yet,but status is still pending.
Contributor License Agreement was signed yet,but status is still pending.
That's because the email of your commits does not match the email of your GitHub account. You may need to readjust your commit with your GitHub-associated email.
Contributor License Agreement was signed yet,but status is still pending.
That's because the email of your commits does not match the email of your GitHub account. You may need to readjust your commit with your GitHub-associated email.
Thanks for your reply.Everythins is ok,beside review.Cloud you review the code?
@sczyh30 Cloud you review the code?Thandks.
@sczyh30 Have a look,please.
@sczyh30 Code has been changed,Thanks for your review ~ btw, is your profile picture murongziying in the legend of sword and fairy 4?
@sczyh30 help!
How time flies! @sczyh30 go on please ~
@sczyh30 @jasonjoo2010 @brotherlu-xcq @LearningGp help~
@sczyh30 @jasonjoo2010 @brotherlu-xcq @LearningGp help~
@sczyh30 @jasonjoo2010 @brotherlu-xcq @LearningGp help~
cc @LearningGp
cc ,please @LearningGp
What ‘s’ the average processing time for pr?
@LearningGp cc
@sczyh30 @jasonjoo2010 @brotherlu-xcq @LearningGp help~
@panxiaojun233 @jasonjoo2010 @brotherlu-xcq @LearningGp Have a look,please.
Shall I change traget branch to 1.8?
Shall I change traget branch to 1.8?
I'm sorry to have shelved this PR for so long, please change the target branch to 1.8 if it is convenient.
