arextest / arex-agent-java

Lightweight Java agent for traffic capture and replay, enhancing testing and debugging.
https://doc.arextest.com/
Apache License 2.0
416 stars 82 forks source link

feat: support ignore transform by configured type or classloader prefixes #514

Closed shichanglin5 closed 5 days ago

shichanglin5 commented 1 week ago

fix https://github.com/arextest/arex-agent-java/issues/513

shichanglin5 commented 1 week ago

@lucas-myx please take a review

shichanglin5 commented 1 week ago

@mr3 新的 commit 增加了配置 ignoreTypePrefixes,简化了 ArexIgnoreMatcher 构造,再 review 下呢

mr3 commented 6 days ago

@mr3 新的 commit 增加了配置 ignoreTypePrefixes,简化了 ArexIgnoreMatcher 构造,再 review 下呢

Copy that. I'll check it out later.

mr3 commented 6 days ago

@shichanglin5 Thanks very much for the PR!

I've made a small change, type and classloader are now unified as prefix configuration, please complete the unit tests for the changed codes, and we will merge this PR for regression testing later.

shichanglin5 commented 6 days ago

@shichanglin5 Thanks very much for the PR!

I've made a small change, type and classloader are now unified as prefix configuration, please complete the unit tests for the changed codes, and we will merge this PR for regression testing later.

好的,建议把ot agent 相关忽略默认加上,ot agent 非常通用,降低同时挂载场景时的接入和理解成本

sonarcloud[bot] commented 6 days ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
84.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

mr3 commented 6 days ago

@shichanglin5 LGTM.