bookstack/bookstack
bookstack/bookstack
Watch
44
261
Fork
You've already forked bookstack
 32
Code Issues 699 Pull requests 8 Releases 240 Activity Actions 26

New Crowdin translations by GitHub Action #6211

Closed
Chinmay_21 wants to merge 1 commit from l10n_development into development
pull from: l10n_development
merge into: bookstack:development
Conversation 1 Commits 1 Files changed 210 +1467 -11
Chinmay_21 commented 2026-09-13 10:16:24 +02:00
First-time contributor
Copy link

Details

Fixes #6208.

This PR fixes an issue where saving a BookStack page containing certain special characters as inline code results in a 403 Forbidden error.

The affected characters can be used legitimately in file-name documentation, so they should not prevent a page from being saved.

Changes

  • Fixed the handling of the affected special characters when saving page content.
  • Added a regression test to prevent the issue from occurring again.

Checklist

  • I have read the BookStack community rules.
  • This PR does not feature significant use of LLM/AI generation as per the community rules above.
## Details Fixes #6208. This PR fixes an issue where saving a BookStack page containing certain special characters as inline code results in a `403 Forbidden` error. The affected characters can be used legitimately in file-name documentation, so they should not prevent a page from being saved. ### Changes * Fixed the handling of the affected special characters when saving page content. * Added a regression test to prevent the issue from occurring again. ## Checklist * [x] I have read the [BookStack community rules](https://www.bookstackapp.com/about/community-rules/). * [x] This PR does not feature significant use of LLM/AI generation as per the community rules above.
New Crowdin translations by GitHub Action
All checks were successful
test-migrations / build (8.2) (pull_request) Successful in 1m47s
Details
analyse-php / build (pull_request) Successful in 2m51s
Details
lint-php / build (pull_request) Successful in 2m58s
Details
test-migrations / build (8.3) (pull_request) Successful in 3m17s
Details
test-migrations / build (8.4) (pull_request) Successful in 2m14s
Details
test-migrations / build (8.5) (pull_request) Successful in 1m52s
Details
test-php / build (8.2) (pull_request) Successful in 7m37s
Details
test-php / build (8.3) (pull_request) Successful in 8m45s
Details
test-php / build (8.5) (pull_request) Successful in 7m55s
Details
test-php / build (8.4) (pull_request) Successful in 8m35s
Details
2536443ff0
danb commented 2026-09-13 14:18:18 +02:00
Owner
Copy link

Hi @Chinmay_21,

Not sure if something has gone wrong here.
This does not address #6208, but is just a PR for our existing translation branch.

Will therefore close this off.

Hi @Chinmay_21, Not sure if something has gone wrong here. This does not address #6208, but is just a PR for our existing translation branch. Will therefore close this off.
danb closed this pull request 2026-09-13 14:18:19 +02:00
All checks were successful
test-migrations / build (8.2) (pull_request) Successful in 1m47s
Details
analyse-php / build (pull_request) Successful in 2m51s
Details
lint-php / build (pull_request) Successful in 2m58s
Details
test-migrations / build (8.3) (pull_request) Successful in 3m17s
Details
test-migrations / build (8.4) (pull_request) Successful in 2m14s
Details
test-migrations / build (8.5) (pull_request) Successful in 1m52s
Details
test-php / build (8.2) (pull_request) Successful in 7m37s
Details
test-php / build (8.3) (pull_request) Successful in 8m45s
Details
test-php / build (8.5) (pull_request) Successful in 7m55s
Details
test-php / build (8.4) (pull_request) Successful in 8m35s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Focus: A11y
Accessibility

  
Focus: Admin/Meta

   
Focus: Authentication

   
Focus: Back-End

   
Focus: Database

   
Focus: Design & UX

   
Focus: Editor - Markdown

   
Focus: Editor - WYSIWYG

   
Focus: Export System

   
Focus: Front-End

   
Focus: Translations

   
Focus: View Customization

   
Is: Docs Update

   
Is: Enhancement

   
Is: Priority

   
Is: Security

   
Is: Upstream
  
Status
Blocked

  
Status
Open to discussion

  
Status
Out of scope

  
Status
Pending Validation

  
Type
API Request

  
Type
Bug Report

  
Type
Feature Request

  
Type
Happy feedback

  
Type
Maintenance

  
Type
Question

  
Type
Support

No labels
Focus: A11y
Focus: Admin/Meta
Focus: Authentication
Focus: Back-End
Focus: Database
Focus: Design & UX
Focus: Editor - Markdown
Focus: Editor - WYSIWYG
Focus: Export System
Focus: Front-End
Focus: Translations
Focus: View Customization
Is: Docs Update
Is: Enhancement
Is: Priority
Is: Security
Is: Upstream
Status
Blocked
Status
Open to discussion
Status
Out of scope
Status
Pending Validation
Type
API Request
Type
Bug Report
Type
Feature Request
Type
Happy feedback
Type
Maintenance
Type
Question
Type
Support
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
bookstack/bookstack!6211
No description provided.