From: Micah Cowan <micah@cowan.name>
Date: Sat, 12 Apr 2008 19:57:25 +0000 (-0700)
Subject: Merging in the rest of Alex's bundle.
X-Git-Tag: v1.13~430^2~2
X-Git-Url: http://sjero.net/git/?p=wget;a=commitdiff_plain;h=4e288c5db5242cf5851e1d113824eae5d5fb0e02;hp=0e7917cda7c06b223745fb9ff95b2f129eee55de

Merging in the rest of Alex's bundle.
---

diff --git a/doc/ChangeLog b/doc/ChangeLog
index 343f09f7..c5614d4a 100644
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -1,3 +1,8 @@
+2008-04-11  Micah Cowan  <micah@cowan.name>
+
+	* wget.texi <Contributors>: Added Julien Buty, Alexander
+	Dergachev, and Rabin Vincent.
+
 2008-03-24  Micah Cowan  <micah@cowan.name>
 
 	* wget.texi <Types of Fields>: Mentioned various caveats in the
diff --git a/doc/wget.texi b/doc/wget.texi
index cd97bcf0..583c7fcf 100644
--- a/doc/wget.texi
+++ b/doc/wget.texi
@@ -3770,6 +3770,7 @@ Paul Bludov,
 Daniel Bodea,
 Mark Boyns,
 John Burden,
+Julien Buty,
 Wanderlei Cavassin,
 Gilles Cedoc,
 Tim Charron,
@@ -3785,6 +3786,7 @@ Andreas Damm,
 Ahmon Dancy,
 Andrew Davison,
 Bertrand Demiddelaer,
+Alexander Dergachev,
 Andrew Deryabin,
 Ulrich Drepper,
 Marc Duponcheel,
@@ -3938,6 +3940,7 @@ Philipp Thomas,
 Mauro Tortonesi,
 Dave Turner,
 Gisle Vanem,
+Rabin Vincent,
 Russell Vincent,
 @iftex
 @v{Z}eljko Vrba,
diff --git a/src/ChangeLog b/src/ChangeLog
index d320f150..a2881858 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -3,6 +3,13 @@
 	* utils.c (aprintf): Now we are setting limits (1 Mb) for text
 	buffer when we use non-C99 vsnprintf.
 	
+2008-04-11  Micah Cowan  <micah@cowan.name>
+
+	* ftp.c (getftp, ftp_loop_internal): Don't append to an existing
+	.listing when --continue is used.  Fixes bug #22825. Thanks to
+	Rabin Vincent <rabin@rab.in> for pointing the way with a
+	suggested fix!
+
 2008-04-10  Alexander Dergachev  <cy6erbr4in@gmail.com>
 
 	* xmalloc.c, xmalloc.h (memfatal): Now exported; accepts an
diff --git a/src/ftp.c b/src/ftp.c
index 115cc5c4..03210321 100644
--- a/src/ftp.c
+++ b/src/ftp.c
@@ -918,7 +918,7 @@ Error in server response, closing control connection.\n"));
       if (opt.backups)
         rotate_backups (con->target);
 
-      if (restval)
+      if (restval && !(con->cmd & DO_LIST))
         fp = fopen (con->target, "ab");
       else if (opt.noclobber || opt.always_rest || opt.timestamping || opt.dirstruct
                || opt.output_document)
@@ -1141,7 +1141,9 @@ ftp_loop_internal (struct url *u, struct fileinfo *f, ccon *con)
         }
 
       /* Decide whether or not to restart.  */
-      if (opt.always_rest
+      if (con->cmd & DO_LIST)
+        restval = 0;
+      else if (opt.always_rest
           && stat (locf, &st) == 0
           && S_ISREG (st.st_mode))
         /* When -c is used, continue from on-disk size.  (Can't use
diff --git a/tests/ChangeLog b/tests/ChangeLog
index b4e49381..e6972b50 100644
--- a/tests/ChangeLog
+++ b/tests/ChangeLog
@@ -1,3 +1,8 @@
+2008-04-10  Micah Cowan  <micah@cowan.name>
+
+	* Makefile.in, Test-proxy-auth-basic.px: Added a test for
+	accessing password-protected URLs through a proxy.
+
 2008-01-25  Micah Cowan  <micah@cowan.name>
 
 	* Makefile.am: Updated copyright year.
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 61efdd91..224cb9cc 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -46,6 +46,7 @@ run-unit-tests: unit-tests$(EXEEXT)
 	./unit-tests$(EXEEXT)
 
 run-px-tests: WgetTest.pm
+	$(PERLRUN) $(srcdir)/Test-proxy-auth-basic.px && echo && echo
 	$(PERLRUN) $(srcdir)/Test-auth-basic.px && echo && echo
 	$(PERLRUN) $(srcdir)/Test-c-full.px && echo && echo
 	$(PERLRUN) $(srcdir)/Test-c-partial.px && echo && echo
diff --git a/tests/Test-proxy-auth-basic.px b/tests/Test-proxy-auth-basic.px
new file mode 100755
index 00000000..5566e22f
--- /dev/null
+++ b/tests/Test-proxy-auth-basic.px
@@ -0,0 +1,48 @@
+#!/usr/bin/perl -w
+
+use strict;
+
+use HTTPTest;
+
+
+###############################################################################
+
+my $wholefile = "You're all authenticated.\n";
+
+# code, msg, headers, content
+my %urls = (
+    '/needs-auth.txt' => {
+        auth_method => 'Basic',
+        user => 'fiddle-dee-dee',
+        passwd => 'Dodgson',
+        code => "200",
+        msg => "You want fries with that?",
+        headers => {
+            "Content-type" => "text/plain",
+        },
+        content => $wholefile,
+    },
+);
+
+my $cmdline = $WgetTest::WGETPATH . " --debug --user=fiddle-dee-dee --password=Dodgson"
+    . " -e http_proxy=localhost:8080 http://no.such.domain/needs-auth.txt";
+
+my $expected_error_code = 0;
+
+my %expected_downloaded_files = (
+    'needs-auth.txt' => {
+        content => $wholefile,
+    },
+);
+
+###############################################################################
+
+my $the_test = HTTPTest->new (name => "Test-auth-basic",
+                              input => \%urls, 
+                              cmdline => $cmdline, 
+                              errcode => $expected_error_code, 
+                              output => \%expected_downloaded_files);
+exit $the_test->run();
+
+# vim: et ts=4 sw=4
+