Skip to content

Commit

Permalink
Ability to configure HTTP header limit.
Browse files Browse the repository at this point in the history
  • Loading branch information
MattBlissett committed Nov 21, 2018
1 parent 300e4ef commit a3098c3
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
<jdkLevel>1.8</jdkLevel>

<curator.version>2.9.1</curator.version>
<service-discovery.version>0.1</service-discovery.version>
<service-discovery.version>0.2</service-discovery.version>
<maven-core.version>3.3.9</maven-core.version>
<jackson.version>1.9.13</jackson.version>
<jettyVersion>9.3.8.v20160314</jettyVersion>
Expand Down
10 changes: 8 additions & 2 deletions src/main/java/org/gbif/jetty/HttpConnectorFactory.java
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
package org.gbif.jetty;

import org.eclipse.jetty.server.Connector;
import org.eclipse.jetty.server.HttpConfiguration;
import org.eclipse.jetty.server.HttpConnectionFactory;
import org.eclipse.jetty.server.Server;
import org.eclipse.jetty.server.ServerConnector;

Expand Down Expand Up @@ -96,11 +98,15 @@ public void setServer(Server server) {
* Builds an instance of a Connector using the values provider by the HttpConnectorFactory instance.
*/
public Connector build() {
final ServerConnector httpConnector = new ServerConnector(server);
final HttpConfiguration httpConfiguration = new HttpConfiguration();
httpConfiguration.setRequestHeaderSize(maxRequestHeaderSize);

final HttpConnectionFactory httpConnectionFactory = new HttpConnectionFactory(httpConfiguration);

final ServerConnector httpConnector = new ServerConnector(server, httpConnectionFactory);
httpConnector.setPort(port);
httpConnector.setIdleTimeout(idleTimeout);
httpConnector.setName(name);
return httpConnector;
}

}
5 changes: 5 additions & 0 deletions src/main/java/org/gbif/jetty/ServerFactory.java
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,13 @@ public Server build(ServiceConfiguration configuration) {
private static Connector[] buildConnectors(ServiceConfiguration configuration, Server server) {
final HttpConnectorFactory appConnectorFactory = HttpConnectorFactory.application(server);
appConnectorFactory.setPort(configuration.getHttpPort());
if (configuration.getMaxRequestHeaderSize() != null) {
appConnectorFactory.setMaxRequestHeaderSize(configuration.getMaxRequestHeaderSize());
}

final HttpConnectorFactory adminConnectorFactory = HttpConnectorFactory.admin(server);
adminConnectorFactory.setPort(configuration.getHttpAdminPort());

return new Connector[] {appConnectorFactory.build(), adminConnectorFactory.build()};
}

Expand Down

0 comments on commit a3098c3

Please sign in to comment.