Skip to content
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

Add remote ip #499

Merged
merged 6 commits into from
Nov 18, 2020
Merged

Add remote ip #499

merged 6 commits into from
Nov 18, 2020

Conversation

sxci
Copy link
Contributor

@sxci sxci commented Nov 13, 2020

当链接超时,在异常信息中添加 ip 端口信息。

以前:
com.qiniu.common.QiniuException: java.net.SocketTimeoutException: connect timed out
Caused by: java.io.IOException: java.net.SocketTimeoutException: connect timed out

修改后:
com.qiniu.common.QiniuException: java.net.SocketTimeoutException: connect timed out on rs.qbox.me/115.231.97.60:80
Caused by: java.io.IOException: java.net.SocketTimeoutException: connect timed out on rs.qbox.me/115.231.97.60:80
Caused by: java.io.IOException: java.net.SocketTimeoutException: connect timed out

@codecov
Copy link

codecov bot commented Nov 13, 2020

Codecov Report

Merging #499 (7e5dfac) into master (3be435d) will decrease coverage by 0.15%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #499      +/-   ##
============================================
- Coverage     54.47%   54.32%   -0.16%     
- Complexity      704      750      +46     
============================================
  Files            94       96       +2     
  Lines          4677     4898     +221     
  Branches        509      512       +3     
============================================
+ Hits           2548     2661     +113     
- Misses         1860     1976     +116     
+ Partials        269      261       -8     
Impacted Files Coverage Δ Complexity Δ
src/main/java/com/qiniu/http/Client.java 68.13% <100.00%> (+4.30%) 30.00 <0.00> (ø)
...main/java/com/qiniu/qvs/model/StaticLiveRoute.java 30.43% <0.00%> (-14.01%) 7.00% <0.00%> (+3.00%) ⬇️
src/main/java/com/qiniu/http/Response.java 55.04% <0.00%> (-4.59%) 27.00% <0.00%> (-7.00%)
src/main/java/com/qiniu/qvs/NameSpaceManager.java 80.00% <0.00%> (-4.22%) 15.00% <0.00%> (+5.00%) ⬇️
.../main/java/com/qiniu/storage/FixBlockUploader.java 72.47% <0.00%> (-3.53%) 45.00% <0.00%> (-2.00%)
src/main/java/com/qiniu/qvs/model/Template.java 35.33% <0.00%> (-0.49%) 37.00% <0.00%> (+18.00%) ⬇️
src/main/java/com/qiniu/qvs/model/Device.java 57.89% <0.00%> (ø) 9.00% <0.00%> (?%)
src/main/java/com/qiniu/qvs/DeviceManager.java 87.50% <0.00%> (ø) 10.00% <0.00%> (?%)
src/main/java/com/qiniu/qvs/model/NameSpace.java 38.55% <0.00%> (+4.00%) 24.00% <0.00%> (+9.00%)
... and 2 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 3be435d...7e5dfac. Read the comment docs.

@sxci sxci merged commit de4b6e0 into master Nov 18, 2020
@sxci sxci deleted the add_remote_ip branch November 18, 2020 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants