Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit d60ab76

Browse files
committed
Silence perlcritic warnings in new libpq tests
Per buildfarm member 'koel'.
1 parent 91044ae commit d60ab76

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/test/libpq_encryption/t/001_negotiate_encryption.pl

+3-2
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,8 @@
135135
|| die "copying server.crt: $!";
136136
copy "$certdir/server-cn-only.key", "$pgdata/server.key"
137137
|| die "copying server.key: $!";
138-
chmod(0600, "$pgdata/server.key");
138+
chmod(0600, "$pgdata/server.key")
139+
or die "failed to change permissions on server keys: $!";
139140

140141
# Start with SSL disabled.
141142
$node->append_conf('postgresql.conf', "ssl = off\n");
@@ -181,7 +182,7 @@ BEGIN
181182

182183
# Only accept SSL connections from $servercidr. Our tests don't depend on this
183184
# but seems best to keep it as narrow as possible for security reasons.
184-
open my $hba, '>', "$pgdata/pg_hba.conf";
185+
open my $hba, '>', "$pgdata/pg_hba.conf" or die $!;
185186
print $hba qq{
186187
# TYPE DATABASE USER ADDRESS METHOD OPTIONS
187188
local postgres localuser trust

0 commit comments

Comments
 (0)