-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
feature: support send authentication msg #2825
Conversation
Signed-off-by: slievrly <slievrly@163.com>
Codecov Report
@@ Coverage Diff @@
## develop #2825 +/- ##
=============================================
- Coverage 50.45% 50.42% -0.03%
- Complexity 3108 3115 +7
=============================================
Files 593 594 +1
Lines 19571 19613 +42
Branches 2427 2431 +4
=============================================
+ Hits 9874 9890 +16
- Misses 8702 8724 +22
- Partials 995 999 +4
|
Signed-off-by: slievrly <slievrly@163.com>
Signed-off-by: slievrly <slievrly@163.com>
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.
Please solve the conflict.
Signed-off-by: slievrly <slievrly@163.com> # Conflicts: # core/src/main/java/io/seata/core/rpc/netty/TmNettyRemotingClient.java
Codecov Report
@@ Coverage Diff @@
## develop #2825 +/- ##
=============================================
- Coverage 51.09% 51.04% -0.05%
- Complexity 3253 3257 +4
=============================================
Files 608 609 +1
Lines 19979 20021 +42
Branches 2494 2498 +4
=============================================
+ Hits 10208 10220 +12
- Misses 8760 8783 +23
- Partials 1011 1018 +7 |
if (null != extraData) { | ||
sb.append(extraData); | ||
} | ||
if (transactionServiceGroup != null && !transactionServiceGroup.isEmpty()) { |
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.
Do you need to add EXTRA_DATA_SPLIT_CHAR for this block?
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.
这块是否需要添加EXTRA_DATA_SPLIT_CHAR来进行区分?
Signed-off-by: slievrly <slievrly@163.com>
Signed-off-by: slievrly <slievrly@163.com> # Conflicts: # spring/src/main/java/io/seata/spring/annotation/GlobalTransactionScanner.java
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
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
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 for @ph3636
Signed-off-by: slievrly slievrly@163.com
Ⅰ. Describe what this PR did
feature: support send authentication msg
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews