Bug Report: Trailing space of page title adds unnecessary - to URL #443

New Issue

OVERLORD · 2026-02-04T20:03:47+03:00

OVERLORD commented

2026-02-04 20:03:47 +03:00

Originally created by @mackcoding on GitHub (Sep 26, 2017).

BookStack Version: Latest (all versions)
PHP Version: N/A
MySQL Version: N/A

Expected Behavior

URL should be:
https://demo.bookstackapp.com/books/bookstack-user-guide/page/test

Actual Behavior

Actual url is
https://demo.bookstackapp.com/books/bookstack-user-guide/page/test-

Browsing to https://demo.bookstackapp.com/books/bookstack-user-guide/page/test causes an error.

This can be fixed by trimming whitespace.

Originally created by @mackcoding on GitHub (Sep 26, 2017). * BookStack Version: Latest (all versions) * PHP Version: N/A * MySQL Version: N/A ##### Expected Behavior URL should be: https://demo.bookstackapp.com/books/bookstack-user-guide/page/test ##### Actual Behavior Actual url is https://demo.bookstackapp.com/books/bookstack-user-guide/page/test- Browsing to https://demo.bookstackapp.com/books/bookstack-user-guide/page/test causes an error. This can be fixed by trimming whitespace.

OVERLORD closed this issue

2026-02-04 20:04:01 +03:00

OVERLORD commented

2026-02-04 20:04:08 +03:00

@ssddanbrown commented on GitHub (Sep 30, 2017):

Hi @mackcoding, Thanks for reporting.

Thought Laravel did this out of the box but looks like i needed to add in the middleware. Fixed as of aca6de49b0. Will be in next bugfix release.

@ssddanbrown commented on GitHub (Sep 30, 2017): Hi @mackcoding, Thanks for reporting. Thought Laravel did this out of the box but looks like i needed to add in the middleware. Fixed as of aca6de49b0324c9b1cf48a79908f6d3ad2fd0d5a. Will be in next bugfix release.

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#443