diff options
author | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-05-04 12:19:15 +0000 |
---|---|---|
committer | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-05-04 12:19:15 +0000 |
commit | 6eb9c5a5657d1fe77b55cc261450f3538d35a94d (patch) | |
tree | 657d8194422a5daccecfd42d654b8a245ef7b4c8 /src/test/authentication | |
parent | Initial commit. (diff) | |
download | postgresql-13-6eb9c5a5657d1fe77b55cc261450f3538d35a94d.tar.xz postgresql-13-6eb9c5a5657d1fe77b55cc261450f3538d35a94d.zip |
Adding upstream version 13.4.upstream/13.4upstream
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'src/test/authentication')
-rw-r--r-- | src/test/authentication/.gitignore | 2 | ||||
-rw-r--r-- | src/test/authentication/Makefile | 23 | ||||
-rw-r--r-- | src/test/authentication/README | 26 | ||||
-rw-r--r-- | src/test/authentication/t/001_password.pl | 124 | ||||
-rw-r--r-- | src/test/authentication/t/002_saslprep.pl | 106 |
5 files changed, 281 insertions, 0 deletions
diff --git a/src/test/authentication/.gitignore b/src/test/authentication/.gitignore new file mode 100644 index 0000000..871e943 --- /dev/null +++ b/src/test/authentication/.gitignore @@ -0,0 +1,2 @@ +# Generated by test suite +/tmp_check/ diff --git a/src/test/authentication/Makefile b/src/test/authentication/Makefile new file mode 100644 index 0000000..124d1d0 --- /dev/null +++ b/src/test/authentication/Makefile @@ -0,0 +1,23 @@ +#------------------------------------------------------------------------- +# +# Makefile for src/test/authentication +# +# Portions Copyright (c) 1996-2020, PostgreSQL Global Development Group +# Portions Copyright (c) 1994, Regents of the University of California +# +# src/test/authentication/Makefile +# +#------------------------------------------------------------------------- + +subdir = src/test/authentication +top_builddir = ../../.. +include $(top_builddir)/src/Makefile.global + +check: + $(prove_check) + +installcheck: + $(prove_installcheck) + +clean distclean maintainer-clean: + rm -rf tmp_check diff --git a/src/test/authentication/README b/src/test/authentication/README new file mode 100644 index 0000000..a8f27bf --- /dev/null +++ b/src/test/authentication/README @@ -0,0 +1,26 @@ +src/test/authentication/README + +Regression tests for authentication +=================================== + +This directory contains a test suite for authentication. SSL certificate +authentication tests are kept separate, in src/test/ssl/, because they +are more complicated, and are not safe to run in a multi-user system. + + +Running the tests +================= + +NOTE: You must have given the --enable-tap-tests argument to configure. + +Run + make check +or + make installcheck +You can use "make installcheck" if you previously did "make install". +In that case, the code in the installation tree is tested. With +"make check", a temporary installation tree is built from the current +sources and then tested. + +Either way, this test initializes, starts, and stops a test Postgres +cluster. diff --git a/src/test/authentication/t/001_password.pl b/src/test/authentication/t/001_password.pl new file mode 100644 index 0000000..80b4b96 --- /dev/null +++ b/src/test/authentication/t/001_password.pl @@ -0,0 +1,124 @@ +# Set of tests for authentication and pg_hba.conf. The following password +# methods are checked through this test: +# - Plain +# - MD5-encrypted +# - SCRAM-encrypted +# This test can only run with Unix-domain sockets. + +use strict; +use warnings; +use PostgresNode; +use TestLib; +use Test::More; +if (!$use_unix_sockets) +{ + plan skip_all => + "authentication tests cannot run without Unix-domain sockets"; +} +else +{ + plan tests => 13; +} + + +# Delete pg_hba.conf from the given node, add a new entry to it +# and then execute a reload to refresh it. +sub reset_pg_hba +{ + my $node = shift; + my $hba_method = shift; + + unlink($node->data_dir . '/pg_hba.conf'); + $node->append_conf('pg_hba.conf', "local all all $hba_method"); + $node->reload; + return; +} + +# Test access for a single role, useful to wrap all tests into one. +sub test_role +{ + my $node = shift; + my $role = shift; + my $method = shift; + my $expected_res = shift; + my $status_string = 'failed'; + + $status_string = 'success' if ($expected_res eq 0); + + local $Test::Builder::Level = $Test::Builder::Level + 1; + + my $res = $node->psql('postgres', undef, extra_params => [ '-U', $role, '-w' ]); + is($res, $expected_res, + "authentication $status_string for method $method, role $role"); + return; +} + +# Initialize master node +my $node = get_new_node('master'); +$node->init; +$node->start; + +# Create 3 roles with different password methods for each one. The same +# password is used for all of them. +$node->safe_psql('postgres', + "SET password_encryption='scram-sha-256'; CREATE ROLE scram_role LOGIN PASSWORD 'pass';" +); +$node->safe_psql('postgres', + "SET password_encryption='md5'; CREATE ROLE md5_role LOGIN PASSWORD 'pass';" +); +$ENV{"PGPASSWORD"} = 'pass'; + +# For "trust" method, all users should be able to connect. +reset_pg_hba($node, 'trust'); +test_role($node, 'scram_role', 'trust', 0); +test_role($node, 'md5_role', 'trust', 0); + +# For plain "password" method, all users should also be able to connect. +reset_pg_hba($node, 'password'); +test_role($node, 'scram_role', 'password', 0); +test_role($node, 'md5_role', 'password', 0); + +# For "scram-sha-256" method, user "scram_role" should be able to connect. +reset_pg_hba($node, 'scram-sha-256'); +test_role($node, 'scram_role', 'scram-sha-256', 0); +test_role($node, 'md5_role', 'scram-sha-256', 2); + +# For "md5" method, all users should be able to connect (SCRAM +# authentication will be performed for the user with a SCRAM secret.) +reset_pg_hba($node, 'md5'); +test_role($node, 'scram_role', 'md5', 0); +test_role($node, 'md5_role', 'md5', 0); + +# Tests for channel binding without SSL. +# Using the password authentication method; channel binding can't work +reset_pg_hba($node, 'password'); +$ENV{"PGCHANNELBINDING"} = 'require'; +test_role($node, 'scram_role', 'scram-sha-256', 2); +# SSL not in use; channel binding still can't work +reset_pg_hba($node, 'scram-sha-256'); +$ENV{"PGCHANNELBINDING"} = 'require'; +test_role($node, 'scram_role', 'scram-sha-256', 2); + +# Test .pgpass processing; but use a temp file, don't overwrite the real one! +my $pgpassfile = "${TestLib::tmp_check}/pgpass"; + +delete $ENV{"PGPASSWORD"}; +delete $ENV{"PGCHANNELBINDING"}; +$ENV{"PGPASSFILE"} = $pgpassfile; + +unlink($pgpassfile); +append_to_file($pgpassfile, qq! +# This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. +*:*:postgres:scram_role:pass:this is not part of the password. +!); +chmod 0600, $pgpassfile or die; + +reset_pg_hba($node, 'password'); +test_role($node, 'scram_role', 'password from pgpass', 0); +test_role($node, 'md5_role', 'password from pgpass', 2); + +append_to_file($pgpassfile, qq! +*:*:*:md5_role:p\\ass +!); + +test_role($node, 'md5_role', 'password from pgpass', 0); diff --git a/src/test/authentication/t/002_saslprep.pl b/src/test/authentication/t/002_saslprep.pl new file mode 100644 index 0000000..32d4e43 --- /dev/null +++ b/src/test/authentication/t/002_saslprep.pl @@ -0,0 +1,106 @@ +# Test password normalization in SCRAM. +# +# This test can only run with Unix-domain sockets. + +use strict; +use warnings; +use PostgresNode; +use TestLib; +use Test::More; +if (!$use_unix_sockets) +{ + plan skip_all => + "authentication tests cannot run without Unix-domain sockets"; +} +else +{ + plan tests => 12; +} + +# Delete pg_hba.conf from the given node, add a new entry to it +# and then execute a reload to refresh it. +sub reset_pg_hba +{ + my $node = shift; + my $hba_method = shift; + + unlink($node->data_dir . '/pg_hba.conf'); + $node->append_conf('pg_hba.conf', "local all all $hba_method"); + $node->reload; + return; +} + +# Test access for a single role, useful to wrap all tests into one. +sub test_login +{ + my $node = shift; + my $role = shift; + my $password = shift; + my $expected_res = shift; + my $status_string = 'failed'; + + $status_string = 'success' if ($expected_res eq 0); + + $ENV{"PGPASSWORD"} = $password; + my $res = $node->psql('postgres', undef, extra_params => [ '-U', $role ]); + is($res, $expected_res, + "authentication $status_string for role $role with password $password" + ); + return; +} + +# Initialize master node. Force UTF-8 encoding, so that we can use non-ASCII +# characters in the passwords below. +my $node = get_new_node('master'); +$node->init(extra => [ '--locale=C', '--encoding=UTF8' ]); +$node->start; + +# These tests are based on the example strings from RFC4013.txt, +# Section "3. Examples": +# +# # Input Output Comments +# - ----- ------ -------- +# 1 I<U+00AD>X IX SOFT HYPHEN mapped to nothing +# 2 user user no transformation +# 3 USER USER case preserved, will not match #2 +# 4 <U+00AA> a output is NFKC, input in ISO 8859-1 +# 5 <U+2168> IX output is NFKC, will match #1 +# 6 <U+0007> Error - prohibited character +# 7 <U+0627><U+0031> Error - bidirectional check + +# Create test roles. +$node->safe_psql( + 'postgres', + "SET password_encryption='scram-sha-256'; +SET client_encoding='utf8'; +CREATE ROLE saslpreptest1_role LOGIN PASSWORD 'IX'; +CREATE ROLE saslpreptest4a_role LOGIN PASSWORD 'a'; +CREATE ROLE saslpreptest4b_role LOGIN PASSWORD E'\\xc2\\xaa'; +CREATE ROLE saslpreptest6_role LOGIN PASSWORD E'foo\\x07bar'; +CREATE ROLE saslpreptest7_role LOGIN PASSWORD E'foo\\u0627\\u0031bar'; +"); + +# Require password from now on. +reset_pg_hba($node, 'scram-sha-256'); + +# Check that #1 and #5 are treated the same as just 'IX' +test_login($node, 'saslpreptest1_role', "I\xc2\xadX", 0); +test_login($node, 'saslpreptest1_role', "\xe2\x85\xa8", 0); + +# but different from lower case 'ix' +test_login($node, 'saslpreptest1_role', "ix", 2); + +# Check #4 +test_login($node, 'saslpreptest4a_role', "a", 0); +test_login($node, 'saslpreptest4a_role', "\xc2\xaa", 0); +test_login($node, 'saslpreptest4b_role', "a", 0); +test_login($node, 'saslpreptest4b_role', "\xc2\xaa", 0); + +# Check #6 and #7 - In PostgreSQL, contrary to the spec, if the password +# contains prohibited characters, we use it as is, without normalization. +test_login($node, 'saslpreptest6_role', "foo\x07bar", 0); +test_login($node, 'saslpreptest6_role', "foobar", 2); + +test_login($node, 'saslpreptest7_role', "foo\xd8\xa71bar", 0); +test_login($node, 'saslpreptest7_role', "foo1\xd8\xa7bar", 2); +test_login($node, 'saslpreptest7_role', "foobar", 2); |