Re: [PATCH] Pull general SASL framework out of SCRAM

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [PATCH] Pull general SASL framework out of SCRAM
Дата
Msg-id YOpwhZFOpC0UUQaB@paquier.xyz
обсуждение исходный текст
Ответ на Re: [PATCH] Pull general SASL framework out of SCRAM  (Jacob Champion <pchampion@vmware.com>)
Ответы Re: [PATCH] Pull general SASL framework out of SCRAM
Список pgsql-hackers
On Fri, Jul 09, 2021 at 11:31:48PM +0000, Jacob Champion wrote:
> On Thu, 2021-07-08 at 16:27 +0900, Michael Paquier wrote:
>> +     *    outputlen: The length (0 or higher) of the client response buffer,
>> +     *               invalid if output is NULL.
>
> nitpick: maybe "ignored" instead of "invalid"?

Thanks, applied as 44bd012 after using your suggestion.

Another thing I noticed after more review is that the check in
fe-auth.c to make sure that a message needs to be generated if the
exchange is not completed yet has no need to depend on "success", only
"done".
--
Michael

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Alexander Korotkov
Дата:
Сообщение: Re: unnesting multirange data types
Следующее
От: Thomas Munro
Дата:
Сообщение: Re: pgbench logging broken by time logic changes