webkit report: unhelpful diagnosis message when wkhtmltopdf exists with error status

Bug #1003819 reported by Alexandre Fayolle - camptocamp
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fix Committed
Alexandre Fayolle - camptocamp

Bug Description

Alexis Delattre reported an issue on c2c-financial-addons which also affects the report_webkit module (https://bugs.launchpad.net/c2c-financial-addons/+bug/1003563).

There is another issue that I tackled on the c2c module which is that the code was using subprocess.call with stderr=PIPE which is something that must never be done as it can lead to a deadlock in the application (see http://docs.python.org/library/subprocess.html?highlight=subprocess#using-the-subprocess-module)

Related branches

Changed in openobject-addons:
assignee: nobody → Nicolas Bessi - Camptocamp (nbessi-c2c)
Changed in openobject-addons:
status: New → Confirmed
Changed in openobject-addons:
assignee: Nicolas Bessi - Camptocamp (nbessi-c2c) → Alexandre Fayolle @ camptocamp (alexandre-fayolle-c2c)
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

Hello Ravish,

I have the merge requests approved by Nicolas. How do we proceed to get this merged in 6.0, 6.1 and trunk?

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Alexandre,

You have provided a MP for all version 6.0, 6.1 and trunk Our Core team will review it and merge.


Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

hello, any news here?

Changed in openobject-addons:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers