From 911ac1685c573e05fa72a87d25a2f15ca88767eb Mon Sep 17 00:00:00 2001
From: Harald Hope <smxi@users.noreply.github.com>
Date: Tue, 12 Sep 2017 17:56:28 -0700
Subject: [PATCH] New version, tarball. Bug fix. Debug data collector using
 Perl requires explicitly setting Passive => 1 (true) for some systems and
 firewall configurations.

This corrects a failure to upload issue I experienced for a test remote
system that had a different firewall configuration than the dev system has.
---
 inxi           |  9 +++++----
 inxi.changelog | 18 ++++++++++++++++++
 2 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/inxi b/inxi
index d24f88d..b6f1b6f 100755
--- a/inxi
+++ b/inxi
@@ -2,8 +2,8 @@
 ########################################################################
 SELF_NAME='inxi' 
 # don't quote the following, parsers grab these too
-SELF_VERSION=2.3.38
-SELF_DATE=2017-09-07
+SELF_VERSION=2.3.39
+SELF_DATE=2017-09-12
 SELF_PATCH=00
 ########################################################################
 ####  SPECIAL THANKS
@@ -2185,7 +2185,7 @@ debug_data_collector()
 			ls_sys 2
 			ls_sys 3
 			ls_sys 4
-			# note, this generates more lines than the full sys parsing, so only use if requird
+			# note, this generates more lines than the full sys parsing, so only use if required
 			# ls_sys 5 
 			touch $sys_data_file
 			if type -p perl &>/dev/null;then
@@ -2400,7 +2400,8 @@ upload_debugger_data()
 	$pass = "anonymous\@techpatterns.com";
 	$dir = "incoming";
 	$fpath = $ENV{debugger_file};
-	$ftp = Net::FTP->new($host, Debug => 0);
+	# NOTE: important: must explicitly set to passive true/1
+	$ftp = Net::FTP->new($host, Debug => 0, Passive => 1);
 	$ftp->login($user, $pass) || die $ftp->message;
 	$ftp->binary();
 	$ftp->cwd($dir);
diff --git a/inxi.changelog b/inxi.changelog
index cc00b72..9a361ff 100644
--- a/inxi.changelog
+++ b/inxi.changelog
@@ -1,3 +1,21 @@
+=====================================================================================
+Version: 2.3.39
+Patch Version: 00
+Script Date: 2017-09-12
+-----------------------------------
+Changes:
+-----------------------------------
+New version, tarball. Bug fix. Debug data collector using Perl requires
+explicitly setting Passive => 1 (true) for some systems and firewall 
+configurations.
+
+This corrects a failure to upload issue I experienced for a test remote
+system that had a different firewall configuration than the dev system has.
+
+
+-----------------------------------
+-- Harald Hope - Tue, 12 Sep 2017 17:53:11 -0700
+
 =====================================================================================
 Version: 2.3.38
 Patch Version: 00