Align default keepAlive with gRPC defaults

This changes:
* default keepAliveTime on server to 2 hours (to align with gRPC default)
* default keepAliveTime on client to 5 minutes to align with default permitKeepAliveTime on server (cannot be lower)

Author: cfredri4

grpc-client-spring-boot-autoconfigure/src/main/java/net/devh/boot/grpc/client/config/GrpcChannelProperties.java

@@ -37,7 +37,6 @@
 import io.grpc.NameResolverProvider;
 import io.grpc.internal.GrpcUtil;
 import io.grpc.netty.shaded.io.grpc.netty.NettyChannelBuilder;
-import io.grpc.netty.shaded.io.grpc.netty.NettyServerBuilder;
 import io.grpc.netty.shaded.io.netty.handler.ssl.SslContextBuilder;
 import lombok.EqualsAndHashCode;
 import lombok.ToString;
@@ -177,7 +176,7 @@ public void setEnableKeepAlive(final Boolean enableKeepAlive) {
 
     @DurationUnit(ChronoUnit.SECONDS)
     private Duration keepAliveTime;
-    private static final Duration DEFAULT_KEEP_ALIVE_TIME = Duration.of(60, ChronoUnit.SECONDS);
+    private static final Duration DEFAULT_KEEP_ALIVE_TIME = Duration.of(5, ChronoUnit.MINUTES);
 
     /**
      * Gets the default delay before we send a keepAlive.
@@ -191,13 +190,14 @@ public Duration getKeepAliveTime() {
     }
 
     /**
-     * The default delay before we send a keepAlives. Defaults to {@code 60s}. Default unit {@link ChronoUnit#SECONDS
-     * SECONDS}. Please note that shorter intervals increase the network burden for the server.
+     * The default delay before we send a keepAlives. Defaults to {@code 5min}. Default unit {@link ChronoUnit#SECONDS
+     * SECONDS}. Please note that shorter intervals increase the network burden for the server. Cannot be lower than
+     * permitKeepAliveTime on server (default 5min).
      *
      * @param keepAliveTime The new default delay before sending keepAlives, or null to use the fallback.
      *
      * @see #setEnableKeepAlive(Boolean)
-     * @see NettyServerBuilder#keepAliveTime(long, TimeUnit)
+     * @see NettyChannelBuilder#keepAliveTime(long, TimeUnit)
      */
     public void setKeepAliveTime(final Duration keepAliveTime) {
         this.keepAliveTime = keepAliveTime;
@@ -227,7 +227,7 @@ public Duration getKeepAliveTimeout() {
      * @param keepAliveTimeout The default timeout for a keepAlives ping request.
      *
      * @see #setEnableKeepAlive(Boolean)
-     * @see NettyServerBuilder#keepAliveTimeout(long, TimeUnit)
+     * @see NettyChannelBuilder#keepAliveTimeout(long, TimeUnit)
      */
     public void setKeepAliveTimeout(final Duration keepAliveTimeout) {
         this.keepAliveTimeout = keepAliveTimeout;

grpc-server-spring-boot-autoconfigure/src/main/java/net/devh/boot/grpc/server/config/GrpcServerProperties.java

@@ -113,7 +113,7 @@ public class GrpcServerProperties {
     private boolean enableKeepAlive = false;
 
     /**
-     * The default delay before we send a keepAlives. Defaults to {@code 60s}. Default unit {@link ChronoUnit#SECONDS
+     * The default delay before we send a keepAlives. Defaults to {@code 2h}. Default unit {@link ChronoUnit#SECONDS
      * SECONDS}.
      *
      * @see #setEnableKeepAlive(boolean)
@@ -123,7 +123,7 @@ public class GrpcServerProperties {
      * @return The default delay before sending keepAlives.
      */
     @DurationUnit(ChronoUnit.SECONDS)
-    private Duration keepAliveTime = Duration.of(60, ChronoUnit.SECONDS);
+    private Duration keepAliveTime = Duration.of(2, ChronoUnit.HOURS);
 
     /**
      * The default timeout for a keepAlives ping request. Defaults to {@code 20s}. Default unit