On Wed, Mar 12, 2025 at 3:16 PM Jacob Champion <jacob.champ...@enterprisedb.com> wrote: > I missed PAM_CONV, sorry. I'm worried about the sendAuthRequest() > being done there; it doesn't seem safe to potentially ereport(ERROR) > and longjmp through a PAM call stack? But I'll switch those over to > something safe or else drop that part of the patch.
PAM aside... Michael, what's your level of enthusiasm for the rest of this patch? I was confidently, embarrassingly wrong about how CheckPAMAuth worked, and it makes me think I need to put this down and take a completely new crack at it in 19. Thanks, --Jacob