[PR #5086] [CLOSED] Fix draw.io/diagrams.net "export" function #6450

Closed
opened 2026-02-05 10:32:34 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/5086
Author: @vincentbernat
Created: 6/30/2024
Status: Closed

Base: developmentHead: fix/drawio-export


📝 Commits (1)

  • e3113cb Fix draw.io/diagrams.net "export" function

📊 Changes

2 files changed (+2 additions, -1 deletions)

View changed files

📝 app/Util/CspService.php (+1 -0)
📝 tests/SecurityHeaderTest.php (+1 -1)

📄 Description

Exporting a diagram is done using the "blob:" scheme. This is not allowed by the current CSP. Add it automatically when we have draw.io/diagrams.net integration enabled.

Fix #4710


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/5086 **Author:** [@vincentbernat](https://github.com/vincentbernat) **Created:** 6/30/2024 **Status:** ❌ Closed **Base:** `development` ← **Head:** `fix/drawio-export` --- ### 📝 Commits (1) - [`e3113cb`](https://github.com/BookStackApp/BookStack/commit/e3113cb32919ee49d53c3b1147801dfa1b0ee4bf) Fix draw.io/diagrams.net "export" function ### 📊 Changes **2 files changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Util/CspService.php` (+1 -0) 📝 `tests/SecurityHeaderTest.php` (+1 -1) </details> ### 📄 Description Exporting a diagram is done using the "blob:" scheme. This is not allowed by the current CSP. Add it automatically when we have draw.io/diagrams.net integration enabled. Fix #4710 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:32:34 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6450