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

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [PATCH] Pull general SASL framework out of SCRAM
Дата
Msg-id YNZ5Bqq4GvhNvbp0@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  (Jacob Champion <pchampion@vmware.com>)
Список pgsql-hackers
On Fri, Jun 25, 2021 at 11:40:33PM +0000, Jacob Champion wrote:
> I can definitely move it (into, say, auth-sasl.c?). I'll probably do
> that in a second commit, though, since keeping it in place during the
> refactor makes the review easier IMO.

auth-sasl.c is a name consistent with the existing practice.

> Can do. Does libpq-int-sasl.h work as a filename? This should not be
> exported to applications.

I would still with the existing naming used by fe-gssapi-common.h, so
that would be fe-auth-sasl.c and fe-auth-sasl.h, with the header
remaining internal.  Not strongly wedded to this name, of course, that
just seems consistent.
--
Michael

Вложения

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

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: [patch] remove strver's leftover from error message in Solution.pm
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: Some incorrect logs in TAP tests of pgbench