Merge pull request #843 from mw9/feature/bugfix

Bugfix: HTTPSocks.pm - Binary OR used by mistake - Logical OR needed
LMS-Community · Dec 12, 2022 · 9d57bcf · 9d57bcf
2 parents d02dc32 + 3c19e28
commit 9d57bcf
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/Slim/Networking/Async/HTTP.pm b/Slim/Networking/Async/HTTP.pm
@@ -133,7 +133,6 @@ sub new_socket {
 
 			my %args = @_;
 
-			# Failed. Try again with an explicit SNI.
 			$args{SSL_hostname} //= $args{Host};
 			$args{SSL_verify_mode} //= Net::SSLeay::VERIFY_NONE() if $prefs->get('insecureHTTPS');
 

diff --git a/Slim/Networking/Async/Socket/HTTPSocks.pm b/Slim/Networking/Async/Socket/HTTPSocks.pm
@@ -13,7 +13,7 @@ sub new {
 	# change PeerAddr to proxy (no deepcopy needed)
 	my %params = %args;
 	$params{PeerAddr} = $args{ProxyAddr};
-	$params{PeerPort} = $args{ProxyPort} | 1080;
+	$params{PeerPort} = $args{ProxyPort} || 1080;
 	$params{Blocking} => 1;
 
 	# and connect parent's class to it (better block)