Skip to content

Commit a227857

Browse files
stigtspxdg
authored andcommitted
Fix incorrect env var name for verify_SSL default
The variable to override the verify_SSL default differed slightly in the documentation from what was checked for in the code. This commit makes the code use `PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT` as documented, instead of `PERL_HTTP_TINY_INSECURE_BY_DEFAULT` which was missing `SSL_`
1 parent 6bee932 commit a227857

File tree

3 files changed

+18
-18
lines changed

3 files changed

+18
-18
lines changed

lib/HTTP/Tiny.pm

+1-1
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ sub _verify_SSL_default {
149149
my ($self) = @_;
150150
# Check if insecure default certificate verification behaviour has been
151151
# changed by the user by setting PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1
152-
return (($ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} || '') eq '1') ? 0 : 1;
152+
return (($ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} || '') eq '1') ? 0 : 1;
153153
}
154154

155155
sub _set_proxies {

t/180_verify_SSL.t

+15-15
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ use lib 't';
77

88
use HTTP::Tiny;
99

10-
delete $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT};
10+
delete $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT};
1111

1212
{
1313
my $ht = HTTP::Tiny->new();
@@ -53,54 +53,54 @@ delete $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT};
5353
}
5454

5555
{
56-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "1";
56+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "1";
5757
my $ht = HTTP::Tiny->new();
58-
is($ht->verify_SSL, 0, "PERL_HTTP_TINY_INSECURE_BY_DEFAULT=1 changes verify_SSL default to 0");
58+
is($ht->verify_SSL, 0, "PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1 changes verify_SSL default to 0");
5959
}
6060

6161
{
62-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "0";
62+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "0";
6363
my $ht = HTTP::Tiny->new();
64-
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_INSECURE_BY_DEFAULT=0 keeps verify_SSL default at 1");
64+
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=0 keeps verify_SSL default at 1");
6565
}
6666

6767
{
68-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "False";
68+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "False";
6969
my $ht = HTTP::Tiny->new();
70-
is($ht->verify_SSL, 1, "Unsupported PERL_HTTP_TINY_INSECURE_BY_DEFAULT=False keeps verify_SSL default at 1");
70+
is($ht->verify_SSL, 1, "Unsupported PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=False keeps verify_SSL default at 1");
7171
}
7272

7373
{
74-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "1";
74+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "1";
7575
my $ht = HTTP::Tiny->new(verify_SSL=>1);
76-
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_INSECURE_BY_DEFAULT=1 does not override verify_SSL attribute set to 1");
76+
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1 does not override verify_SSL attribute set to 1");
7777
}
7878

7979
{
80-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "1";
80+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "1";
8181
my $ht = HTTP::Tiny->new(
8282
verify_SSL => 1,
8383
verify_ssl => 1
8484
);
85-
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_INSECURE_BY_DEFAULT=1, verify_SSL=>1 and verify_ssl=>1 sets 1");
85+
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1, verify_SSL=>1 and verify_ssl=>1 sets 1");
8686
}
8787

8888
{
89-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "1";
89+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "1";
9090
my $ht = HTTP::Tiny->new(
9191
verify_SSL => 1,
9292
verify_ssl => 0
9393
);
94-
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_INSECURE_BY_DEFAULT=1, verify_SSL=>1 and verify_ssl=>0 sets 1");
94+
is($ht->verify_SSL, 1, "PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1, verify_SSL=>1 and verify_ssl=>0 sets 1");
9595
}
9696

9797
{
98-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = "1";
98+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = "1";
9999
my $ht = HTTP::Tiny->new(
100100
verify_SSL => 0,
101101
verify_ssl => 0
102102
);
103-
is($ht->verify_SSL, 0, "PERL_HTTP_TINY_INSECURE_BY_DEFAULT=1, verify_SSL=>0 and verify_ssl=>0 sets 0");
103+
is($ht->verify_SSL, 0, "PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1, verify_SSL=>0 and verify_ssl=>0 sets 0");
104104
}
105105

106106

t/210_live_ssl.t

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ BEGIN {
1818
}
1919
use HTTP::Tiny;
2020

21-
delete $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT};
21+
delete $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT};
2222

2323
plan skip_all => 'Only run for $ENV{AUTOMATED_TESTING}'
2424
unless $ENV{AUTOMATED_TESTING};
@@ -65,7 +65,7 @@ test_ssl('https://mozilla-modern.badssl.com/' => {
6565
});
6666

6767
{
68-
local $ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} = 1;
68+
local $ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} = 1;
6969
test_ssl('https://wrong.host.badssl.com/' => {
7070
host => 'wrong.host.badssl.com',
7171
pass => { verify_SSL => 0 },

0 commit comments

Comments
 (0)