Skip to content

Commit

Permalink
FELIX-5787 : Wrong port 433 instread of 443 used as default port for …
Browse files Browse the repository at this point in the history
…https

git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@1823743 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
cziegeler committed Feb 10, 2018
1 parent c2899c4 commit 1cc1298
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ public ObjectClassDefinition getObjectClassDefinition( String id, String locale
return null;
}

final ArrayList<AttributeDefinition> adList = new ArrayList<AttributeDefinition>();
final ArrayList<AttributeDefinition> adList = new ArrayList<>();

adList.add(new AttributeDefinitionImpl(JettyConfig.FELIX_HOST,
"Host Name",
Expand Down Expand Up @@ -91,8 +91,8 @@ public ObjectClassDefinition getObjectClassDefinition( String id, String locale

adList.add(new AttributeDefinitionImpl(JettyConfig.HTTPS_PORT,
"HTTPS Port",
"Port to listen on for HTTPS requests. Defaults to 433.",
433,
"Port to listen on for HTTPS requests. Defaults to 443.",
443,
bundle.getBundleContext().getProperty(JettyConfig.HTTPS_PORT)));

adList.add(new AttributeDefinitionImpl(JettyConfig.FELIX_KEYSTORE,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ public class JettyConfigTest
props.put("org.osgi.service.http.port.secure", "*");
this.config.update(props);
assertTrue(this.config.getHttpPort() != 8080);
assertTrue(this.config.getHttpsPort() != 433);
assertTrue(this.config.getHttpsPort() != 443);
}

@Test public void testGetRandomPortZero() throws Exception
Expand Down

0 comments on commit 1cc1298

Please sign in to comment.