Skip to content

Commit

Permalink
feat: add ssl to postgres
Browse files Browse the repository at this point in the history
  • Loading branch information
sprutton1 committed Jan 2, 2024
1 parent 5027100 commit 6ba837d
Show file tree
Hide file tree
Showing 30 changed files with 1,111 additions and 19 deletions.
78 changes: 67 additions & 11 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,8 @@ regex = "1.8.1"
remain = "0.2.8"
reqwest = { version = "0.11.17", default-features = false, features = ["rustls-tls", "json", "multipart"] }
ring = "=0.17.5" # Upgrading this is possible, but a pain, so we don't want to pick up every new minor version (see: https://github.com/facebook/buck2/commit/91af40b66960d003067c3d241595fb53d1e636c8)
rustls = { version = "0.21.1" } # pinned because ring above depends on it
rustls-pemfile = { version = "2.0.0" }
rust-s3 = { version = "0.34.0-rc4", default-features = false, features = ["tokio-rustls-tls"] }
sea-orm = { version = "0.12.0", features = ["sqlx-postgres", "runtime-tokio-rustls", "macros", "with-chrono", "debug-print"] }
self-replace = "1.3.7"
Expand All @@ -130,6 +132,7 @@ test-log = { version = "0.2.11", default-features = false, features = ["trace"]
thiserror = "1.0.40"
tokio = { version = "1.28.0", features = ["full"] }
tokio-postgres = { version = "0.7.8", features = ["runtime", "with-chrono-0_4", "with-serde_json-1"] }
tokio-postgres-rustls = { version = "0.10.0" }
tokio-serde = { version = "0.8.0", features = ["json"] }
tokio-stream = "0.1.14"
tokio-test = "0.4.2"
Expand All @@ -147,6 +150,7 @@ url = { version = "2.3.1", features = ["serde"] }
uuid = { version = "1.3.2", features = ["serde", "v4"] }
vfs = "0.10.0"
vfs-tar = { version = "0.4.0", features = ["mmap"] }
webpki-roots = { version = "0.25.3" }
y-sync = { version = "0.4.0", features = ["net"] }
yrs = { version = "0.17.2" }

Expand Down
1 change: 1 addition & 0 deletions bin/module-index/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ rust_binary(
resources = {
"dev.jwt_signing_public_key.pem": "//config/keys:dev.jwt_signing_public_key.pem",
"prod.jwt_signing_public_key.pem": "//config/keys:prod.jwt_signing_public_key.pem",
"dev.postgres.root.crt": "//config/keys:dev.postgres.root.crt",
# "pkgs_path": "//pkgs:pkgs",
},
)
Expand Down
14 changes: 14 additions & 0 deletions bin/module-index/src/args.rs
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,14 @@ pub(crate) struct Args {
#[arg(long, env)]
pub(crate) pg_password: Option<String>,

/// PostgreSQL connection certification path
#[arg(long)]
pub(crate) pg_cert_path: Option<String>,

/// PostgreSQL connection certification base64 string
#[arg(long)]
pub(crate) pg_cert_base64: Option<String>,

/// The address and port to bind the HTTP server to [example: 0.0.0.0:80]
#[arg(long, env)]
pub(crate) socket_addr: Option<String>,
Expand Down Expand Up @@ -105,6 +113,12 @@ impl TryFrom<Args> for Config {
if let Some(password) = args.pg_password {
config_map.set("pg.password", password);
}
if let Some(cert) = args.pg_cert_path {
config_map.set("pg.certificate_path", cert);
}
if let Some(cert) = args.pg_cert_base64 {
config_map.set("pg.certificate_base64", cert);
}
if let Some(socket_addr) = args.socket_addr {
config_map.set("socket_addr", socket_addr);
}
Expand Down
3 changes: 2 additions & 1 deletion bin/pinga/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ rust_binary(
resources = {
"dev.encryption.key": "//lib/cyclone-server:dev.encryption.key",
"dev.donkey.key": "//lib/dal:dev.donkey.key",
"dev.postgres.root.crt": "//config/keys:dev.postgres.root.crt",
},
)

Expand All @@ -32,4 +33,4 @@ nix_omnibus_pkg(
name = "omnibus",
pkg_name = "pinga",
build_dep = "//bin/pinga:pinga",
)
)
14 changes: 14 additions & 0 deletions bin/pinga/src/args.rs
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,14 @@ pub(crate) struct Args {
#[arg(long)]
pub(crate) pg_user: Option<String>,

/// PostgreSQL connection certification path
#[arg(long)]
pub(crate) pg_cert_path: Option<String>,

/// PostgreSQL connection certification base64 string
#[arg(long)]
pub(crate) pg_cert_base64: Option<String>,

/// NATS connection URL [example: demo.nats.io]
#[arg(long)]
pub(crate) nats_url: Option<String>,
Expand Down Expand Up @@ -101,6 +109,12 @@ impl TryFrom<Args> for Config {
if let Some(user) = args.pg_user {
config_map.set("pg.user", user);
}
if let Some(cert) = args.pg_cert_path {
config_map.set("pg.certificate_path", cert);
}
if let Some(cert) = args.pg_cert_base64 {
config_map.set("pg.certificate_base64", cert);
}
if let Some(url) = args.nats_url {
config_map.set("nats.url", url);
}
Expand Down
3 changes: 2 additions & 1 deletion bin/sdf/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ rust_binary(
"dev.jwt_signing_public_key.pem": "//config/keys:dev.jwt_signing_public_key.pem",
"prod.jwt_signing_public_key.pem": "//config/keys:prod.jwt_signing_public_key.pem",
"dev.encryption.key": "//lib/cyclone-server:dev.encryption.key",
"dev.postgres.root.crt": "//config/keys:dev.postgres.root.crt",
"dev.donkey.key": "//lib/dal:dev.donkey.key",
"pkgs_path": "//pkgs:pkgs",
},
Expand All @@ -38,4 +39,4 @@ nix_omnibus_pkg(
name = "omnibus",
pkg_name = "sdf",
build_dep = "//bin/sdf:sdf",
)
)
14 changes: 14 additions & 0 deletions bin/sdf/src/args.rs
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,14 @@ pub(crate) struct Args {
#[arg(long)]
pub(crate) pg_user: Option<String>,

/// PostgreSQL connection certification path
#[arg(long)]
pub(crate) pg_cert_path: Option<String>,

/// PostgreSQL connection certification base64 string
#[arg(long)]
pub(crate) pg_cert_base64: Option<String>,

/// NATS connection URL [example: demo.nats.io]
#[arg(long)]
pub(crate) nats_url: Option<String>,
Expand Down Expand Up @@ -119,6 +127,12 @@ impl TryFrom<Args> for Config {
if let Some(user) = args.pg_user {
config_map.set("pg.user", user);
}
if let Some(cert) = args.pg_cert_path {
config_map.set("pg.certificate_path", cert);
}
if let Some(cert) = args.pg_cert_base64 {
config_map.set("pg.certificate_base64", cert);
}
if let Some(migration_mode) = args.migration_mode {
config_map.set("migration_mode", migration_mode);
}
Expand Down
2 changes: 2 additions & 0 deletions component/postgres/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ docker_image(
"multiple-database-support.sh": ".",
"pganalyze-collector.conf.sample": ".",
"postgresql-additions.conf": ".",
"server.key": ".",
"server.crt": ".",
},
build_args = {
"BASE_VERSION": "14.5-bullseye",
Expand Down
5 changes: 5 additions & 0 deletions component/postgres/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,11 @@ COPY multiple-database-support.sh /docker-entrypoint-initdb.d/
COPY entrypoint-wrapper.sh /usr/local/bin/entrypoint-wrapper.sh
COPY pganalyze-collector.conf.sample /etc/pganalyze-collector.conf.sample
COPY postgresql-additions.conf /etc/postgresql-additions.conf
COPY server.key /var/lib/postgresql/
COPY server.crt /var/lib/postgresql/

RUN chown root:ssl-cert /var/lib/postgresql/server.key && chmod 740 /var/lib/postgresql/server.key
RUN chown root:ssl-cert /var/lib/postgresql/server.crt && chmod 740 /var/lib/postgresql/server.crt

RUN chmod +x /usr/local/bin/entrypoint-wrapper.sh \
&& mkdir -p /etc/postgresql \
Expand Down
5 changes: 4 additions & 1 deletion component/postgres/entrypoint-wrapper.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,10 @@ trap 'kill -TERM $(jobs -p); wait' SIGINT SIGTERM

/usr/local/bin/docker-entrypoint.sh \
"$@" \
-c config_file=/etc/postgresql/postgresql.conf &
-c config_file=/etc/postgresql/postgresql.conf \
-c ssl=on \
-c ssl_cert_file=/var/lib/postgresql/server.crt \
-c ssl_key_file=/var/lib/postgresql/server.key &

if [ -n "${PGANALYZE:-}" ]; then
echo '--- pganalyze enabled'
Expand Down
20 changes: 20 additions & 0 deletions component/postgres/server.crt
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
-----BEGIN CERTIFICATE-----
MIIDNzCCAh+gAwIBAgIIJB180Ufm2cYwDQYJKoZIhvcNAQELBQAwIDEeMBwGA1UE
AxMVbWluaWNhIHJvb3QgY2EgMDIxMTg0MB4XDTI0MDEwMjE4MjI1NloXDTI2MDIw
MTE4MjI1NlowFDESMBAGA1UEAxMJbG9jYWxob3N0MIIBIjANBgkqhkiG9w0BAQEF
AAOCAQ8AMIIBCgKCAQEAo6AmFdhKoE1KWEGHof3mhYgtXxXjYTgE7ux25cpJmg2l
TuQL/AhUkJqTdcRwjQeNZdp7YgnuzxnOCyZWddyRn68fxfAhRUlv0aBMedLrF4/O
aEekRC8hxlLFSATe1w2/dc7EIVsezSlBAHCzwAlpaaWA275LxEuVR+N/+esFxgkw
3qOyutylxEaeeH2ELdV2PW8GuXCQHCISlQa0blaTsBeG8bLJpuUjFeo3vYj7rP3E
dtFoEEJ3tMbd3ku5m5iLubfHlnHDZIY/EF29j3frI7KLRuEhzhOL8F0/U7iLATgj
yit3dZvBQYw/XZqTlDs7/WES+eqFJpaVMhb6QcDhMQIDAQABo4GAMH4wDgYDVR0P
AQH/BAQDAgWgMB0GA1UdJQQWMBQGCCsGAQUFBwMBBggrBgEFBQcDAjAMBgNVHRMB
Af8EAjAAMB8GA1UdIwQYMBaAFAcYAHyM52e+nG6VLw2UjYvSNHFTMB4GA1UdEQQX
MBWCCWxvY2FsaG9zdIIIcG9zdGdyZXMwDQYJKoZIhvcNAQELBQADggEBAJWrwSaz
rT+Q9lMmvW64L+b6r6X4vi3gHhQzZnbfj/7KhNxdbfIromoN0tYjhWrH4Uy+w4Oc
8Mz+KDXqu1XzHMbSSt4w75GnZ2YJuLu6lIbJtX3l3Q1LAai+tZ6FoqIdBIBKe7Ft
tbn8lCYRh0vFtTTbUSWX8IHAd1fnP36OFrvj/pO+4c739jVMkMFShE4QabRPJCBP
zOssFHjfM7cKjbIdwkvWySQK2lVC1iSPPSblLn1r++vxHDWy+rBAZnpB32lthJMY
WKBH0xI1Ry2UhlJexyOH+VhACeKCIsWCFN8oRyPISumhe4dYUCgUA2fV6xfsMR/A
zOF5rW9cqadRYAk=
-----END CERTIFICATE-----
Loading

0 comments on commit 6ba837d

Please sign in to comment.