| From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
|---|---|
| To: | Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> |
| Cc: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Require SSL connection to postgres for oauth |
| Date: | 2026-11-06 13:50:19 |
| Message-ID: | 01CFD582-9679-4010-A4F5-033D2ED9F6BD@yesql.se |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 18 Jun 2026, at 06:54, Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> wrote:
> I added another patch which is only a documentation improvement,
> that's the new 0001. The documentation is very clear about "password"
> being insecure over plaintext connections, but there's no such mention
> for oauth.
I cam absolutely get behind an improvement of the docs for master through 18
for this, ideally in time for the next minors. The attached .txt
(intentionally not a .diff or .patch since I didn't re-send your full patchset)
massages your 0001 to make it more consistent with the rest of the
documentation.
> 0002 and 0003 are the previous patches unchanged.
I am intrerested in raising the security model for postgres connections, but as
mentioned here and elsewhere it does require careful study of common container
environments like Kubernetes which may be impacted. For v19 I don't see it as
a realistic change (and I think it was too late even when initially raised,
even though it was a lot earlier in the cycle than now), but hopefully we can
raise the bar in v20.
--
Daniel Gustafsson
| Attachment | Content-Type | Size |
|---|---|---|
| oauth_tls.diff.txt | text/plain | 2.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2026-11-06 14:21:05 | Re: Emptying oauth_validator_libraries doesn't stop OAuth logins |
| Previous Message | Zsolt Parragi | 2026-11-06 13:42:58 | Re: BGWORKER_INTERRUPTIBLE workers are never restarted after DROP/ALTER/CREATE DATABASE |