Relax configuration requirement for password

We now no longer require the password configuration property to be
able to create a connection.

The password requirements depend on authentication method (client-side)
and trust settings (server-side). The requirement cannot be determined
on the client side only, but rather the requirement is a function of
how the server is configured (i.e. implicit trust, 2FA with Postgres 12).

[resolves #178]

Author: mp911de

src/main/java/io/r2dbc/postgresql/PostgresqlConnectionConfiguration.java

@@ -87,9 +87,7 @@ private PostgresqlConnectionConfiguration(String applicationName, boolean autode
         this.forceBinary = forceBinary;
         this.host = Assert.requireNonNull(host, "host must not be null");
         this.options = options;
-        this.password = sslConfig.getSslMode() != SSLMode.DISABLE
-            ? password
-            : Assert.requireNonNull(password, "password must not be null");
+        this.password = password;
         this.port = port;
         this.schema = schema;
         this.username = Assert.requireNonNull(username, "username must not be null");

src/main/java/io/r2dbc/postgresql/api/PostgresqlStatement.java

@@ -44,70 +44,6 @@ public interface PostgresqlStatement extends Statement {
     @Override
     PostgresqlStatement bind(int index, Object value);
 
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, boolean value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, byte value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, char value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, double value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, float value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, int value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, long value) {
-        return bind(index, (Object) value);
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    default PostgresqlStatement bind(int index, short value) {
-        return bind(index, (Object) value);
-    }
-
     /**
      * {@inheritDoc}
      *

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionConfigurationTest.java

@@ -58,7 +58,6 @@ void configuration() {
             .database("test-database")
             .host("test-host")
             .options(options)
-            .password("test-password")
             .port(100)
             .schema("test-schema")
             .username("test-username")
@@ -71,7 +70,7 @@ void configuration() {
             .hasFieldOrPropertyWithValue("database", "test-database")
             .hasFieldOrPropertyWithValue("host", "test-host")
             .hasFieldOrPropertyWithValue("options", options)
-            .hasFieldOrPropertyWithValue("password", "test-password")
+            .hasFieldOrPropertyWithValue("password", null)
             .hasFieldOrPropertyWithValue("port", 100)
             .hasFieldOrPropertyWithValue("schema", "test-schema")
             .hasFieldOrPropertyWithValue("username", "test-username")
@@ -108,15 +107,6 @@ void constructorNoNoHost() {
             .withMessage("host must not be null");
     }
 
-    @Test
-    void constructorNoPassword() {
-        assertThatIllegalArgumentException().isThrownBy(() -> PostgresqlConnectionConfiguration.builder()
-            .host("test-host")
-            .username("test-username")
-            .build())
-            .withMessage("password must not be null");
-    }
-
     @Test
     void constructorNoUsername() {
         assertThatIllegalArgumentException().isThrownBy(() -> PostgresqlConnectionConfiguration.builder()

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryProviderTest.java

@@ -137,7 +137,16 @@ void supportsSsl() {
     }
 
     @Test
-    void providerShouldparseAndHandleConnectionParameters() {
+    void shouldCreateConnectionFactoryWithoutPassword() {
+        assertThat(this.provider.create(ConnectionFactoryOptions.builder()
+            .option(DRIVER, POSTGRESQL_DRIVER)
+            .option(HOST, "test-host")
+            .option(USER, "test-user")
+            .build())).isNotNull();
+    }
+
+    @Test
+    void providerShouldParseAndHandleConnectionParameters() {
         Map<String, String> expectedOptions = new HashMap<>();
         expectedOptions.put("lock_timeout", "5s");
         expectedOptions.put("statement_timeout", "6000");