HDDS-1235. BaseHttpServer NPE is HTTP policy is HTTPS_ONLY. Contributed by Xiaoyu Yao.

Closes #572

(cherry picked from commit 11db46956c6062a986caf9009d3e36049e5830d4)
This commit is contained in:
Xiaoyu Yao 2019-03-09 09:38:01 -08:00
parent 6465125a5a
commit 49217971b5

View File

@ -65,14 +65,10 @@ public abstract class BaseHttpServer {
public BaseHttpServer(Configuration conf, String name) throws IOException {
this.name = name;
this.conf = conf;
policy = DFSUtil.getHttpPolicy(conf);
if (isEnabled()) {
policy = DFSUtil.getHttpPolicy(conf);
if (policy.isHttpEnabled()) {
this.httpAddress = getHttpBindAddress();
}
if (policy.isHttpsEnabled()) {
this.httpsAddress = getHttpsBindAddress();
}
this.httpAddress = getHttpBindAddress();
this.httpsAddress = getHttpsBindAddress();
HttpServer2.Builder builder = null;
builder = DFSUtil.httpServerTemplateForNNAndJN(conf, this.httpAddress,
this.httpsAddress, name, getSpnegoPrincipal(), getKeytabFile());