Password prompt only once, and before backgrounding.

[wget] / src / main.c

diff --git a/src/main.c b/src/main.c
index 5fb91664ca65e63ec728550e7caa92c385248dc9..d68cdbd6b937db48b44ca3d20b926c88e27cee08 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -56,7 +56,8 @@ as that of the covered work.  */
 #include "http.h"               /* for save_cookies */
 
 #include <getopt.h>
-#include "getpass.h"
+#include <getpass.h>
+#include <quote.h>
 
 #ifndef PATH_SEPARATOR
 # define PATH_SEPARATOR '/'
@@ -681,7 +682,7 @@ static char *
 prompt_for_password (void)
 {
   if (opt.user)
-    printf (_("Password for user \"%s\": "), opt.user);
+    printf (_("Password for user %s: "), quote (opt.user));
   else
     printf (_("Password: "));
   return getpass("");
@@ -919,6 +920,12 @@ WARNING: timestamping does nothing in combination with -O. See the manual\n\
 for details.\n\n"));
           opt.timestamping = false;
         }
+      if (opt.noclobber && file_exists_p(opt.output_document)) 
+           { 
+              /* Check if output file exists; if it does, exit. */
+              logprintf (LOG_VERBOSE, _("File `%s' already there; not retrieving.\n"), opt.output_document);
+              exit(1);
+           }  
     }
 
   if (opt.ask_passwd && opt.passwd)
@@ -940,6 +947,14 @@ for details.\n\n"));
       exit (1);
     }
 
+  if (opt.ask_passwd)
+    {
+      opt.passwd = prompt_for_password ();
+
+      if (opt.passwd == NULL || opt.passwd[0] == '\0')
+        exit (1);
+    }
+
 #ifdef MSDOS
   if (opt.wdebug)
      dbug_init();
@@ -1023,9 +1038,6 @@ for details.\n\n"));
       char *filename = NULL, *redirected_URL = NULL;
       int dt;
 
-      if (opt.ask_passwd)
-        opt.passwd = prompt_for_password ();
-
       if ((opt.recursive || opt.page_requisites)
           && (url_scheme (*t) != SCHEME_FTP || url_uses_proxy (*t)))
         {