HTTPAuthSession close()

core/src/main/java/org/apache/iceberg/rest/auth/AuthSession.java

@@ -30,7 +30,16 @@
 public interface AuthSession extends AutoCloseable {
 
   /** An empty session that does nothing. */
-  AuthSession EMPTY = request -> request;
+  AuthSession EMPTY =
+      new AuthSession() {
+        @Override
+        public HTTPRequest authenticate(HTTPRequest request) {
+          return request;
+        }
+
+        @Override
+        public void close() {}
+      };
 
   /**
    * Authenticates the given request and returns a new request with the necessary authentication.
@@ -44,7 +53,5 @@ public interface AuthSession extends AutoCloseable {
    * the cache, or the cache itself is closed.
    */
   @Override
-  default void close() {
-    // Do nothing
-  }
+  void close();
 }

core/src/main/java/org/apache/iceberg/rest/auth/DefaultAuthSession.java

@@ -46,6 +46,11 @@ default HTTPRequest authenticate(HTTPRequest request) {
         : ImmutableHTTPRequest.builder().from(request).headers(headers).build();
   }
 
+  @Override
+  default void close() {
+    // no resources to close
+  }
+
   static DefaultAuthSession of(HTTPHeaders headers) {
     return ImmutableDefaultAuthSession.builder().headers(headers).build();
   }