Skip to content

Conversation

mhugent
Copy link
Contributor

@mhugent mhugent commented Sep 3, 2025

This PR introduces the new server option QGIS_SERVER_IGNORE_RENDERING_ERRORS . If false (the default), the server throws an error (usually http 500 with text 'internal server error') if a rendering error is reported. If true, the server returns the rendered map/layout even if rendering errors occured

@github-actions github-actions bot added this to the 4.0.0 milestone Sep 3, 2025
Copy link
Contributor

github-actions bot commented Sep 3, 2025

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit 7b6894d)

🪟 Windows builds

Download Windows builds of this PR for testing.
Debug symbols for this build are available here.
(Built from commit 3bf96ab)

🍎 MacOS Qt6 builds

Download MacOS Qt6 builds of this PR for testing.
This installer is not signed, control+click > open the app to avoid the warning
(Built from commit 7b6894d)

@kannes
Copy link
Contributor

kannes commented Sep 4, 2025

This looks very useful! Could you add logging of the ignored errors so that they are not completely hidden?

@mhugent
Copy link
Contributor Author

mhugent commented Sep 11, 2025

That's a good idea, I'm going to add logging to the PR

@mhugent mhugent merged commit 644e968 into qgis:master Sep 12, 2025
33 checks passed
@mhugent mhugent added the Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo. label Sep 12, 2025
@qgis-bot
Copy link
Collaborator

@mhugent
This pull request has been tagged as requiring documentation.

A documentation ticket will be opened at https://github.com/qgis/QGIS-Documentation when this PR is merged.

Please update the description (not the comments) with helpful description and screenshot to help the work from documentors.
Also, any commit having [needs-doc] or [Needs Documentation] in will see its message pushed to the issue, so please be as verbose as you can.

Thank you!

@qgis-bot
Copy link
Collaborator

@mhugent
A documentation ticket has been opened at qgis/QGIS-Documentation#10239
It is your responsibility to visit this ticket and add as much detail as possible for the documentation team to correctly document this change.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants