Skip to content

Fix http tag lock#1809

Open
alanlvle wants to merge 2 commits intosermant-io:developfrom
alanlvle:fix-http-tag-lock
Open

Fix http tag lock#1809
alanlvle wants to merge 2 commits intosermant-io:developfrom
alanlvle:fix-http-tag-lock

Conversation

@alanlvle
Copy link

What type of PR is this?

Feature/Bug/Refactor etc.

What this PR does / why we need it?

Describe it.

Which issue(s) this PR fixes?

Fixes #

Does this PR introduce a user-facing change?

No

Checklist

  • Make sure there is a GitHub_issue related with this PR before you start working on it.
  • Make sure you have squashed your change to one single commit.
  • GitHub Actions works fine in this PR.

lilai23 and others added 2 commits June 30, 2025 20:13
Signed-off-by: lilai <lilai23@foxmail.com>
- 在doAfter和onThrow方法中移除LOCK_MARK线程局部变量
- 防止内存泄漏和线程局部变量污染
- 统一处理OkHttp2.x和OkHttp34插件的资源清理逻辑
@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

❌ Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ion/okhttpv2/interceptors/OkHttp2xInterceptor.java 0.00% 3 Missing ⚠️
...ion/okhttp34/interceptors/OkHttp34Interceptor.java 66.66% 1 Missing ⚠️
Flag Coverage Δ Complexity Δ
unittests 27.23% <33.33%> (+0.08%) 375.00 <1.00> (+34.00)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ Complexity Δ
...ion/okhttp34/interceptors/OkHttp34Interceptor.java 71.87% <66.66%> (-3.99%) 8.00 <1.00> (ø)
...ion/okhttpv2/interceptors/OkHttp2xInterceptor.java 52.77% <0.00%> (-4.80%) 0.00 <0.00> (ø)

... and 26 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants