file object support #2

Activity

1. JazzCore commented on Feb 5, 2013

   

   JazzCore

   on Feb 5, 2013

   Owner

   More actions

   For now output of wkhtmltopdf always goes to stdout, except when you specify --quiet option. Currently if output_path evaluates to False it will append '-' to the end of command ( in command method ). If '-' is at the end of command wkhtmltopdf will pass generated PDF to stdout ( can be used for piping :) ). It is not used in pdfkit at the moment, so this is my mistake for not checking output_path for valid input. So, you can change this behaviour but dont forget to run tests because it can break some of them.

   Moving to communicate: if we use from_string API call we need to pass our input string to stdin of wkhtmltopdf ( wkhtmltopdf requirement ). We dont need to pass something to stdin when using from_file or from_url, except one case: self.source.isFile() and self.css - if we want to add custom CSS to file - it basically reads input file to string and prepends css to it and then passes it to stdin. This is a workaround for a bug in wkhtmltopdf ( look closely in README ). If self.source.isFile() and not self.css - we dont need to pass anything to stdin.

   Yea, it reads only a piece at a time because wkhtmltopdf have a progress-bar in output and this is the only way to properly show it.

   Yes, it will pass resulting PDF document to stdout
2. tomscytale commented on Feb 5, 2013

   

   tomscytale

   on Feb 5, 2013

   ContributorAuthor

   More actions

   great, this looks like it will be very helpful - thanks
3. tomscytale commented on Feb 27, 2013

   

   tomscytale

   on Feb 27, 2013

   ContributorAuthor

   More actions

   some comments about the handling of the output of wkhtmltopdf

   the progress bar and associated details are output to stderr, not stdout

   if, when calling wkhtmltopdf, the output file is specified as '-' the pdf output goes to stdout

   in your call to subprocess.Popen you do not open a pipe to stderr:

   subprocess.Popen(args, stdin=subprocess.PIPE, stdout=subprocess.PIPE)
   

   which means that stderr of wkhtmltopdf goes directly to the stderr of the parent process - i.e. generally it will go directly to the user's terminal. Your code does not handle the stderr (progress bar) at all.
   If you want to handle stderr then the instantiation should be:

   subprocess.Popen(args, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
   

   Also you can't really mix calls to the communicatte() and poll() methods of Popen objects - in this case communicate() consumes all the output of wkhtmltopdf before result() is called. This means that this block actually does nothing:

       if '--quiet' not in args:
           while True:
               if result.poll() is not None:
                   break
               out = result.stdout.read(1).decode('utf-8')
               if out != '':
                   sys.stdout.write(out)
                   sys.stdout.flush()
   

   stuff appears on the console because (as already mentioned) the stderr of wkhtmltopdf is piped to there directly, bypassing pdfkit.
4. JazzCore

   closed this as completedin bc82745on Mar 9, 2013