Skip to content

Commit 1ae2cc2

Browse files
authored
make postgres ssl optional in spec (#2923)
1 parent 0fe2dc7 commit 1ae2cc2

File tree

6 files changed

+5
-6
lines changed

6 files changed

+5
-6
lines changed

airbyte-config/init/src/main/resources/config/STANDARD_SOURCE_DEFINITION/decd338e-5647-4c0b-adf4-da0e75f5a750.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@
22
"sourceDefinitionId": "decd338e-5647-4c0b-adf4-da0e75f5a750",
33
"name": "Postgres",
44
"dockerRepository": "airbyte/source-postgres",
5-
"dockerImageTag": "0.2.6",
5+
"dockerImageTag": "0.2.7",
66
"documentationUrl": "https://hub.docker.com/r/airbyte/source-postgres"
77
}

airbyte-config/init/src/main/resources/seed/source_definitions.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
- sourceDefinitionId: decd338e-5647-4c0b-adf4-da0e75f5a750
2727
name: Postgres
2828
dockerRepository: airbyte/source-postgres
29-
dockerImageTag: 0.2.6
29+
dockerImageTag: 0.2.7
3030
documentationUrl: https://hub.docker.com/r/airbyte/source-postgres
3131
- sourceDefinitionId: cd42861b-01fc-4658-a8ab-5d11d0510f01
3232
name: Recurly

airbyte-integrations/connectors/source-postgres/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ COPY build/distributions/${APPLICATION}*.tar ${APPLICATION}.tar
88

99
RUN tar xf ${APPLICATION}.tar --strip-components=1
1010

11-
LABEL io.airbyte.version=0.2.6
11+
LABEL io.airbyte.version=0.2.7
1212
LABEL io.airbyte.name=airbyte/source-postgres

airbyte-integrations/connectors/source-postgres/src/main/java/io/airbyte/integrations/source/postgres/PostgresSource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public JsonNode toJdbcConfig(JsonNode config) {
9191
config.get("port").asText(),
9292
config.get("database").asText()));
9393

94-
if (config.get("ssl").asBoolean()) {
94+
if (config.has("ssl") && config.get("ssl").asBoolean()) {
9595
additionalParameters.add("ssl=true");
9696
additionalParameters.add("sslmode=require");
9797
}

airbyte-integrations/connectors/source-postgres/src/main/resources/spec.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"$schema": "http://json-schema.org/draft-07/schema#",
55
"title": "Postgres Source Spec",
66
"type": "object",
7-
"required": ["host", "port", "database", "username", "ssl"],
7+
"required": ["host", "port", "database", "username"],
88
"additionalProperties": false,
99
"properties": {
1010
"host": {

airbyte-integrations/connectors/source-postgres/src/test-integration/java/io/airbyte/integrations/io/airbyte/integration_tests/sources/CdcPostgresSourceStandardTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ protected void setup(TestDestinationEnv testEnv) throws Exception {
7272
.put("database", container.getDatabaseName())
7373
.put("username", container.getUsername())
7474
.put("password", container.getPassword())
75-
.put("ssl", false)
7675
.put("replication_method", ImmutableMap.of("replication_slot", SLOT_NAME_BASE))
7776
.build());
7877

0 commit comments

Comments
 (0)