Skip to content

Commit

Permalink
make postgres ssl optional in spec (#2923)
Browse files Browse the repository at this point in the history
  • Loading branch information
jrhizor authored Apr 16, 2021
1 parent 0fe2dc7 commit 1ae2cc2
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@
"sourceDefinitionId": "decd338e-5647-4c0b-adf4-da0e75f5a750",
"name": "Postgres",
"dockerRepository": "airbyte/source-postgres",
"dockerImageTag": "0.2.6",
"dockerImageTag": "0.2.7",
"documentationUrl": "https://hub.docker.com/r/airbyte/source-postgres"
}
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
- sourceDefinitionId: decd338e-5647-4c0b-adf4-da0e75f5a750
name: Postgres
dockerRepository: airbyte/source-postgres
dockerImageTag: 0.2.6
dockerImageTag: 0.2.7
documentationUrl: https://hub.docker.com/r/airbyte/source-postgres
- sourceDefinitionId: cd42861b-01fc-4658-a8ab-5d11d0510f01
name: Recurly
Expand Down
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-postgres/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ COPY build/distributions/${APPLICATION}*.tar ${APPLICATION}.tar

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

LABEL io.airbyte.version=0.2.6
LABEL io.airbyte.version=0.2.7
LABEL io.airbyte.name=airbyte/source-postgres
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ public JsonNode toJdbcConfig(JsonNode config) {
config.get("port").asText(),
config.get("database").asText()));

if (config.get("ssl").asBoolean()) {
if (config.has("ssl") && config.get("ssl").asBoolean()) {
additionalParameters.add("ssl=true");
additionalParameters.add("sslmode=require");
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"$schema": "http://json-schema.org/draft-07/schema#",
"title": "Postgres Source Spec",
"type": "object",
"required": ["host", "port", "database", "username", "ssl"],
"required": ["host", "port", "database", "username"],
"additionalProperties": false,
"properties": {
"host": {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ protected void setup(TestDestinationEnv testEnv) throws Exception {
.put("database", container.getDatabaseName())
.put("username", container.getUsername())
.put("password", container.getPassword())
.put("ssl", false)
.put("replication_method", ImmutableMap.of("replication_slot", SLOT_NAME_BASE))
.build());

Expand Down

0 comments on commit 1ae2cc2

Please sign in to comment.