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.

Sign up for GitHub

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

Jump to bottom

优化host获取，排除127.xxx 0.0.0.0 localhost 等本地IP的注入，从而保证监控中IP的有效性。 #308

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

优化host获取，排除127.xxx 0.0.0.0 localhost 等本地IP的注入，从而保证监控中IP的有效性。 #308

wants to merge 4 commits into from

Conversation

lipangeng
Copy link

优化host获取，排除127.xxx 0.0.0.0 localhost 等本地IP的注入，从而保证监控中IP的有效性。

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.3%) to 41.739% when pulling 877b092 on lipg:master into 2ac5c82 on knightliao:master.

@lipangeng
优化环境变量的注入。使其支持环境变量方式改变配置。
cdc1bb9 
如果通过-D注入的参数为空的话，则检查系统Env变量。并且将.转换为_，如disconf.env转换为disconf_env
@lipangeng
获取本地IP时，增加disconf.host(disconf_host)属性获取，用来指定机器host或者IP。
fd27719 
因为项目依赖问题，无法获取DisClientConfig。遂独立处理该属性。
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.3%) to 41.773% when pulling fd27719 on lipg:master into 2ac5c82 on knightliao:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.4%) to 41.686% when pulling fd27719 on lipg:master into 2ac5c82 on knightliao:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.3%) to 40.782% when pulling 8bd96e1 on lipg:master into 2ac5c82 on knightliao:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lipangeng @coveralls