| From: | "Si, Evan" <evansi(dot)dev(at)gmail(dot)com> |
|---|---|
| To: | surya poondla <suryapoondla4(at)gmail(dot)com> |
| Cc: | Ewan Young <kdbase(dot)hack(at)gmail(dot)com>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Daniel Gustafsson <daniel(at)yesql(dot)se>, "Si, Evan" <evsi(at)amazon(dot)com> |
| Subject: | Re: [PATCH] Clarify that ssl_groups is for any key exchange groups |
| Date: | 2026-08-29 19:52:59 |
| Message-ID: | 8f2d8485-2a29-42af-a981-5c6ef48b4801@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 8/27/26 12:23 PM, surya poondla wrote:
> Thank you for the patch. I was reviewing this as part of the commitfest for
> v20.
Thanks for the review!> v2 patch no longer applies to master.
Apologies, forgot to rebase the patch after it was partially committed.
Nonetheless the cfbot seems to have capably rebased the patch already.
> One minor point in the already-committed docs, "Specifies the named group
> to use for TLS key exchange" is singular while a following sentence says
> multiple groups can be specified.
Makes sense. Attached tweaks to "group(s)" across the board.
Evan
| Attachment | Content-Type | Size |
|---|---|---|
| v3-0001-Clarify-that-ssl_groups-is-for-any-key-exchange-g.patch | text/plain | 5.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2026-08-29 20:10:45 | Re: remove_useless_joins vs. bug #19560 |
| Previous Message | Sehrope Sarkuni | 2026-08-29 19:51:58 | Re: Assorted Win32 error handling fixes (CreateThread, pgwin32_select, pg_usleep) |