issue 3713 - Cannot get the TID using the ConsoleAppender + LogstashE…

[dagmom]

…ncoder + TradeIdJsonProvider in logback.xml

Please answer these questions before submitting pull request

• Why submit this pull request?

• Bug fix

• New feature provided

• Improve performance

• Related issues

issue 3713

---

New feature or improvement

• How to fix?
  LogstashTcpSocketAppender enhance class put TID to ILoggingEvent, but ConsoleAppender don't use ILoggingEvent , so enhance TraceIdJsonProvider class and set traceId

[codecov]

Codecov Report

Merging #5031 into master will decrease coverage by 0.19%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #5031      +/-   ##
============================================
- Coverage     53.15%   52.96%   -0.20%     
+ Complexity     2993     2984       -9     
============================================
  Files           777     1419     +642     
  Lines         19514    30727   +11213     
  Branches       1924     3420    +1496     
============================================
+ Hits          10373    16274    +5901     
- Misses         8288    13648    +5360     
+ Partials        853      805      -48     

Impacted Files	Coverage Δ	Complexity Δ
...log/logback/v1/x/logstash/TraceIdJsonProvider.java	0.00% <0.00%> (ø)	0.00 <0.00> (?)
...k/v1/x/logstash/TraceIdJsonProviderActivation.java	0.00% <0.00%> (ø)	0.00 <0.00> (?)
.../v1/x/logstash/TraceIdJsonProviderInterceptor.java	0.00% <0.00%> (ø)	0.00 <0.00> (?)
...erver/receiver/envoy/MetricServiceGRPCHandler.java	12.16% <0.00%> (-77.84%)	2.00% <0.00%> (ø%)
...er/receiver/envoy/AccessLogServiceGRPCHandler.java	20.75% <0.00%> (-37.15%)	2.00% <0.00%> (ø%)
...skywalking/oap/server/core/alarm/AlarmMessage.java	66.66% <0.00%> (-33.34%)	2.00% <0.00%> (ø%)
...r/cluster/plugin/standalone/StandaloneManager.java	77.77% <0.00%> (-22.23%)	3.00% <0.00%> (-1.00%)
...ient/elasticsearch/ElasticSearchInsertRequest.java	80.00% <0.00%> (-20.00%)	2.00% <0.00%> (ø%)
...ient/elasticsearch/ElasticSearchUpdateRequest.java	80.00% <0.00%> (-20.00%)	2.00% <0.00%> (ø%)
...ticsearch7/client/ElasticSearch7UpdateRequest.java	80.00% <0.00%> (-20.00%)	2.00% <0.00%> (ø%)
... and 966 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b9cbe2...5871fa8. Read the comment docs.

[zhaoyuguang]

The following two problems were found
1 org.apache.commons.lang3.StringUtils class not found

2 why use allArguments[0] getTracingId has no args

[zhaoyuguang]

@dagmom
What kind of demo, The TraceIdJsonProviderInterceptor.java code logic will go here?

[dagmom]

@dagmom
What kind of demo, The TraceIdJsonProviderInterceptor.java code logic will go here?

@zhaoyuguang Maybe async log enhance by AsyncAppenderBaseMethodInterceptor,let me verify that.

[dagmom]

@zhaoyuguang I have confirmed it in the async log

[zhaoyuguang]

Sorry for the delay, I'm will testing asynchrony as soon

[zhaoyuguang]

LGTM @dagmom Please deal This branch is out-of-date with the base branch