Skip to content

Commit

Permalink
fix(postgres-batch-exports): The correct sslmode is disable (#18252)
Browse files Browse the repository at this point in the history
  • Loading branch information
tomasfarias authored Oct 27, 2023
1 parent c090b30 commit ac2e49d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions posthog/temporal/workflows/postgres_batch_export.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ def postgres_connection(inputs):
port=inputs.port,
# The 'hasSelfSignedCert' parameter in the postgres-plugin was provided mainly
# for users of Heroku and RDS. It was used to set 'rejectUnauthorized' to false if a self-signed cert was used.
# Mapping this to sslmode is not straight-forward, but going by Heroku's recommendation (see below) we should use 'no-verify'.
# Mapping this to sslmode is not straight-forward, but going by Heroku's recommendation (see below) we should use 'disable'.
# Reference: https://devcenter.heroku.com/articles/connecting-heroku-postgres#connecting-in-node-js
sslmode="no-verify" if inputs.has_self_signed_cert is True else "prefer",
sslmode="disable" if inputs.has_self_signed_cert is True else "prefer",
)

try:
Expand Down

0 comments on commit ac2e49d

Please sign in to comment.