Skip to content
This repository has been archived by the owner on Jun 3, 2020. It is now read-only.

Zabbix4j http请求 未设置超时时间 #4

Open
honglang1992 opened this issue Nov 3, 2017 · 5 comments
Open

Zabbix4j http请求 未设置超时时间 #4

honglang1992 opened this issue Nov 3, 2017 · 5 comments

Comments

@honglang1992
Copy link

honglang1992 commented Nov 3, 2017

在实际项目中,由于ZabbixApiMethod.sendRequest (line 73) 未设置超市时长。当 zabbix server服务器发生阻塞情况时,一直无返回结果。执行此api的线程就卡在这里了。。

@0312birdzhang
Copy link
Owner

刚坐了两天的车回来。。。明天再看吧,欢迎提交pr

@0312birdzhang
Copy link
Owner

看到了,这地方是用的DefaultHttpClient,太旧了(原作者是用的java1.5),更新一下应该没问题。

@honglang1992
Copy link
Author

我把上周五临时改的上传pr了,仅供参考吧,还没做什么完善。简答说下我的代码功能吧:
1.设为静态,这样不用每个请求new一个 httpclient。
2.httpclient 增加超时时长。

TODO:3.httpclient的相关参数可配置

第三点,我今天开始做。。

@0312birdzhang
Copy link
Owner

👍 👏
等你都改完了我再合并吧
另外第一点,静态的httpclient在多线程请求的时候会不会有问题?🤔

@honglang1992
Copy link
Author

httpClient=HttpClients.custom().setDefaultRequestConfig(defaultRequestConfig).setMaxConnTotal(100).setMaxConnPerRoute(100).build();
默认的是PoolingHttpClientConnectionManager 所以多线程应该没问题

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants