From: | Grzegorz Sampolski <grzsmp(at)gmail(dot)com> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
Cc: | David Fetter <david(at)fetter(dot)org>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pam auth - add rhost item |
Date: | 2016-03-16 11:46:54 |
Message-ID: | 56E947AE.9080207@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi.
Can be, but as you mentioned OS resolver can be configured to not use
dns at all. So much more appropriate will be pam_try_hostname if we want
to be more accurately.
But for me pamusedns, pam_use_hostname or pam_try_hostname all are
correct as either need to use some try to resolve ip address
irrespectively OS resolver use dns or not - I mean getnameinfo() not
give you such information if OS resolver use dns or not.
No to drug the discussion I can change pamusedns to pam_use_hostname if
you prefer.
On 03/16/2016 03:00 AM, Peter Eisentraut wrote:
> On 3/10/16 8:11 AM, Grzegorz Sampolski wrote:
>> In attchment new patch with updated documentation and with small change
>> to coding style as you suggested.
>
> This patch seems fine. I'm not sure about the name "pamusedns" for the
> option, since we use the OS resolver, which might not actually use DNS.
> Maybe something like "pam_use_hostname"?
>
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-03-16 11:57:26 | Re: Parallel Aggregate |
Previous Message | Artur Zakirov | 2016-03-16 11:32:37 | Re: unexpected result from to_tsvector |