From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Kerberos test suite |
Date: | 2018-03-05 19:45:07 |
Message-ID: | a920e02b-fc1a-96d1-528a-a82316015e3d@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2/27/18 00:21, Michael Paquier wrote:
> Thanks. Could you document that on the README please? krb5-user and
> krb5-kdc is a split from Debian. For darwin, are you using macports or
> homebrew? I would assume the later, and it would be nice to precise
> that in the README as well. On Debian you need to install as well
> krb5-admin-server as it includes kadmin.local which the test needs.
> Once I understood that I have been able to run the tests.
Added to README.
> You have forgotten to update ALWAYS_SUBDIRS in src/test/Makefile.
Fixed, and updated to reflect recent changes with PG_TEST_EXTRA etc.
> +my ($stdout, $krb5_version);
> +IPC::Run::run [ 'krb5-config', '--version' ], '>', \$stdout or die
> "could not execute krb5-config";
> +$stdout =~ m/Kerberos 5 release ([0-9]+\.[0-9]+)/ or die "could not get
> Kerberos version";
> +$krb5_version = $1;
> Time for a new routine command_log which executes the command, then
> returns stdout and stderr to the caller?
I didn't do anything about this. Do we have any more places where that
would be needed right now?
> +system_or_bail 'echo secret1 | kinit test1';
> Using IPC::Run stuff would be better here.
done
> @@ -1153,6 +1152,11 @@ sub psql
> $params{on_error_stop} = 1 unless defined $params{on_error_stop};
> $params{on_error_die} = 0 unless defined $params{on_error_die};
>
> + $connstr .= ' host=localhost' if defined $params{tcpip};
> +
> + my @psql_params =
> + ('psql', '-XAtq', '-d', $connstr, '-f', '-');
> This bit I don't like. Wouldn't it be enough to abuse of extra_params
> and use a custom connection string? The last value wins in a psql
> command.
done
Also included feedback from Thomas Munro.
New patch attached.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Tests-for-Kerberos-GSSAPI-authentication.patch | text/plain | 11.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2018-03-05 19:46:38 | Re: [HACKERS] generated columns |
Previous Message | Tom Lane | 2018-03-05 19:34:16 | Re: [HACKERS] user-defined numeric data types triggering ERROR: unsupported type |