Re: [PATCH] Add peer authentication TAP test

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [PATCH] Add peer authentication TAP test
Дата
Msg-id Y4BQO7npF41zV2lU@paquier.xyz
обсуждение исходный текст
Ответ на Re: [PATCH] Add peer authentication TAP test  ("Anton A. Melnikov" <aamelnikov@inbox.ru>)
Ответы Re: [PATCH] Add peer authentication TAP test
Список pgsql-hackers
On Fri, Nov 25, 2022 at 07:56:08AM +0300, Anton A. Melnikov wrote:
> On Windows this test fails with error:
> # connection error: 'psql: error: connection to server at "127.0.0.1", port xxxxx failed:
> # FATAL:  no pg_hba.conf entry for host "127.0.0.1", user "buildfarm", database "postgres", no encryption'
>
> May be disable this test for windows like in 001_password.pl and 002_saslprep.pl?

You are not using MSVC but MinGW, are you?  The buildfarm members with
TAP tests enabled are drongo, fairywren, bowerbord and jacana.  Even
though none of them are running the tests from
src/test/authentication/, this is running on a periodic basis in the
CI, where we are able to skip the test in MSVC already:
postgresql:authentication / authentication/003_peer SKIP 9.73s

So yes, it is plausible that we are missing more safeguards here.

Your suggestion to skip under !$use_unix_sockets makes sense, as not
having unix sockets is not going to work for peer and WIN32 needs SSPI
to be secure with pg_regress.  Where is your test failing?  On the
first $node->psql('postgres') at the beginning of the test?  Just
wondering..
--
Michael

Вложения

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

Предыдущее
От: "Anton A. Melnikov"
Дата:
Сообщение: Re: [PATCH] Add peer authentication TAP test
Следующее
От: Ankit Kumar Pandey
Дата:
Сообщение: Re: Questions regarding distinct operation implementation