Skip to content

[18.0][ADD] report_wkhtmltopdf_local_assets - #1196

Open
marcos-mendez wants to merge 1 commit into
OCA:18.0from
marcos-mendez:18.0-add-report_wkhtmltopdf_local_assets
Open

marcos-mendez wants to merge 1 commit into
OCA:18.0from
marcos-mendez:18.0-add-report_wkhtmltopdf_local_assets

Conversation

@marcos-mendez

Copy link
Copy Markdown

Feed wkhtmltopdf with local files instead of HTTP round-trips to the Odoo workers.

wkhtmltopdf fetches the report assets from the same workers that are waiting for it; once every worker is rendering a PDF the instance deadlocks (odoo/odoo#199880, still open). Static files are referenced from disk, anything else served by the instance is resolved in-process and written to a temporary directory. Local file access stays disabled, only the directories involved are passed with --allow.

Core fix proposed in odoo/odoo#287921.

@OCA-git-bot OCA-git-bot added series:18.0 mod:report_wkhtmltopdf_local_assets Module report_wkhtmltopdf_local_assets labels Sep 11, 2026
@marcos-mendez
marcos-mendez force-pushed the 18.0-add-report_wkhtmltopdf_local_assets branch from 43efd7f to 952a8ab Compare September 14, 2026 13:33
wkhtmltopdf fetches stylesheets, scripts, fonts and images over HTTP from
the workers that are waiting for it. Once every worker is rendering a
PDF, the asset requests queue behind them and the instance stops
answering (odoo/odoo#199880).

Rewrite the report HTML before calling wkhtmltopdf: static files are
referenced from disk, anything else served by the instance is resolved
in-process and written to a temporary directory. Local file access stays
disabled; only the directories involved are passed with --allow.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:report_wkhtmltopdf_local_assets Module report_wkhtmltopdf_local_assets series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants