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

disconf不支持https #158

Closed
guangqan opened this issue Aug 31, 2016 · 2 comments
Closed

disconf不支持https #158

guangqan opened this issue Aug 31, 2016 · 2 comments
Assignees
@knightliao
Labels
bug
Milestone
2.6.36

Comments

@guangqan
Copy link

在com.baidu.disconf.core.common.restful.core.RemoteUrl.RemoteUrl(String, List)里面，以下代码
if (!server.contains("http://")) {
server = "http://" + server;
}
写死了，配置如果没有http://，那么会自动加上。但如果disconf-web是https，则无论如何都支持不了。
@knightliao
Copy link
Owner

这个确实，我来fix

@knightliao knightliao added the bug label Aug 31, 2016
knightliao added a commit that referenced this issue Sep 5, 2016
@knightliao
https://github.com/knightliao/disconf/issues/158
bdba14a
@knightliao knightliao added this to the 2.6.36 milestone Sep 5, 2016
@knightliao knightliao self-assigned this Sep 5, 2016
@knightliao
Copy link
Owner

fix http://disconf.readthedocs.io/zh_CN/latest/others/src/update.html

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

@knightliao knightliao

Labels
bug
Projects
None yet
Milestone
2.6.36
Development

No branches or pull requests
2 participants
@knightliao @guangqan