Skip to content

[type:refactor] refactor log module#4062

Merged
yu199195 merged 9 commits intoapache:masterfrom
moremind:refactor-common-logging
Oct 10, 2022
Merged

[type:refactor] refactor log module#4062
yu199195 merged 9 commits intoapache:masterfrom
moremind:refactor-common-logging

Conversation

@moremind
Copy link
Copy Markdown
Member

@moremind moremind commented Oct 9, 2022

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 10, 2022

Codecov Report

Merging #4062 (cf0e874) into master (1cd41f5) will decrease coverage by 0.13%.
The diff coverage is 39.77%.

@@             Coverage Diff              @@
##             master    #4062      +/-   ##
============================================
- Coverage     70.11%   69.98%   -0.14%     
- Complexity     6748     6758      +10     
============================================
  Files           935      935              
  Lines         25408    25459      +51     
  Branches       2288     2290       +2     
============================================
+ Hits          17816    17817       +1     
- Misses         6202     6245      +43     
- Partials       1390     1397       +7     
Impacted Files Coverage Δ
...u/plugin/logging/common/AbstractLoggingPlugin.java 0.00% <0.00%> (ø)
.../logging/common/body/LoggingServerHttpRequest.java 0.00% <ø> (ø)
...logging/common/body/LoggingServerHttpResponse.java 64.35% <0.00%> (+14.73%) ⬆️
...ogging/common/client/AbstractLogConsumeClient.java 0.00% <0.00%> (ø)
...logging/common/collector/AbstractLogCollector.java 0.00% <0.00%> (ø)
...u/plugin/logging/console/LoggingConsolePlugin.java 47.50% <ø> (ø)
...u/plugin/logging/mask/api/enums/DataMaskEnums.java 100.00% <ø> (ø)
...ugin/logging/mask/api/factory/DataMaskFactory.java 50.00% <ø> (ø)
.../plugin/logging/mask/api/matcher/KeyWordMatch.java 100.00% <ø> (ø)
...n/logging/mask/api/spi/AbstractShenyuDataMask.java 100.00% <ø> (ø)
... and 33 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yu199195 yu199195 added this to the 2.5.1 milestone Oct 10, 2022
@yu199195 yu199195 merged commit cc2a06c into apache:master Oct 10, 2022
ableYang123 pushed a commit to ableYang123/shenyu that referenced this pull request Oct 13, 2022
* refactor log module

* refactor log module

* refactor log module

* refactor log module

* refactor log module

* fix check style

* fix check style

* fix check style

* fix check style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants