On Mon, Feb 24, 2025 at 2:02 PM Jacob Champion
<jacob.champ...@enterprisedb.com> wrote:
> Fair enough. I'll work on a patch to disallow it; best case, no one
> ever complains, and we've pruned an entire configuration from the list
> of things to worry about.

Here goes:

- 0001 fails configuration if the AsynchDNS feature is not built into libcurl.
- 0002 removes EINTR references from the validator documentation and
instead points authors towards our internal Wait APIs.
- 0003 is an optional followup to the const changes from upthread:
there's no need to memcpy() now, and anyone reading the code without
the history might wonder why I chose such a convoluted way to copy a
struct. :D

WDYT?

--Jacob

Attachment: 0002-oauth-Improve-validator-docs-on-interruptibility.patch
Description: Binary data

Attachment: 0001-oauth-Disallow-synchronous-DNS-in-libcurl.patch
Description: Binary data

Attachment: 0003-oauth-Simplify-copy-of-PGoauthBearerRequest.patch
Description: Binary data

Reply via email to