-
Notifications
You must be signed in to change notification settings - Fork 8.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bugfix:fix the problem in the findTargetClass method #3716
bugfix:fix the problem in the findTargetClass method #3716
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3716 +/- ##
=============================================
- Coverage 51.88% 51.87% -0.01%
- Complexity 3533 3534 +1
=============================================
Files 639 639
Lines 21414 21416 +2
Branches 2665 2666 +1
=============================================
- Hits 11111 11110 -1
- Misses 9193 9197 +4
+ Partials 1110 1109 -1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 请在changes文件夹中登记此pr及作者信息到两个1.5.0md中
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Ⅰ. Describe what this PR did
修复获取代理对象类型时, 由于未判断静态类型引起的spring-security框架结合是无法启动的问题
Ⅱ. Does this pull request fix one issue?
fixes #3709
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
在官方提供的例子中
seata-samples->sprngboot-mybatis-sbm-sorage-service项目中
加入maven引用
在启动类的Application类上加入注解 @EnableOAuth2Client
在1.4.2版本下就可以重现#issue 3709的问题。
Ⅴ. Special notes for reviews