mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-07 19:06:05 +03:00
Support table prefixes in database options #2396
Closed
opened 2026-02-05 03:56:37 +03:00 by OVERLORD
·
7 comments
No Branch/Tag Specified
development
l10n_development
further_theme_development
release
llm_only
vectors
v25-11
docker_env
drawio_rendering
user_permissions
ldap_host_failover
svg_image
prosemirror
captcha_example
fix/video-export
v25.12.3
v25.12.2
v25.12.1
v25.12
v25.11.6
v25.11.5
v25.11.4
v24.11.4
v25.11.3
v25.11.2
v25.11.1
v25.11
v25.07.3
v25.07.2
v25.07.1
v25.07
v25.05.2
v25.05.1
v25.05
v25.02.5
v25.02.4
v25.02.3
v25.02.2
v25.02.1
v25.02
v24.12.1
v24.12
v24.10.3
v24.10.2
v24.10.1
v24.10
v24.05.4
v24.05.3
v24.05.2
v24.05.1
v24.05
v24.02.3
v24.02.2
v24.02.1
v24.02
v23.12.3
v23.12.2
v23.12.1
v23.12
v23.10.4
v23.10.3
v23.10.2
v23.10.1
v23.10
v23.08.3
v23.08.2
v23.08.1
v23.08
v23.06.2
v23.06.1
v23.06
v23.05.2
v23.05.1
v23.05
v23.02.3
v23.02.2
v23.02.1
v23.02
v23.01.1
v23.01
v22.11.1
v22.11
v22.10.2
v22.10.1
v22.10
v22.09.1
v22.09
v22.07.3
v22.07.2
v22.07.1
v22.07
v22.06.2
v22.06.1
v22.06
v22.04.2
v22.04.1
v22.04
v22.03.1
v22.03
v22.02.3
v22.02.2
v22.02.1
v22.02
v21.12.5
v21.12.4
v21.12.3
v21.12.2
v21.12.1
v21.12
v21.11.3
v21.11.2
v21.11.1
v21.11
v21.10.3
v21.10.2
v21.10.1
v21.10
v21.08.6
v21.08.5
v21.08.4
v21.08.3
v21.08.2
v21.08.1
v21.08
v21.05.4
v21.05.3
v21.05.2
v21.05.1
v21.05
v21.04.6
v21.04.5
v21.04.4
v21.04.3
v21.04.2
v21.04.1
v21.04
v0.31.8
v0.31.7
v0.31.6
v0.31.5
v0.31.4
v0.31.3
v0.31.2
v0.31.1
v0.31.0
v0.30.7
v0.30.6
v0.30.5
v0.30.4
v0.30.3
v0.30.2
v0.30.1
v0.30.0
v0.29.3
v0.29.2
v0.29.1
v0.29.0
v0.28.3
v0.28.2
v0.28.1
v0.28.0
v0.27.5
v0.27.4
v0.27.3
v0.27.2
v0.27.1
v0.27
v0.26.4
v0.26.3
v0.26.2
v0.26.1
v0.26.0
v0.25.5
v0.25.4
v0.25.3
v0.25.2
v0.25.1
v0.25.0
v0.24.3
v0.24.2
v0.24.1
v0.24.0
v0.23.2
v0.23.1
v0.23.0
v0.22.0
v0.21.0
v0.20.3
v0.20.2
v0.20.1
v0.20.0
v0.19.0
v0.18.5
v0.18.4
v0.18.3
v0.18.2
v0.18.1
v0.18.0
v0.17.4
v0.17.3
v0.17.2
v0.17.1
v0.17.0
v0.16.3
v0.16.2
v0.16.1
v0.16.0
v0.15.3
v0.15.2
v0.15.1
v0.15.0
v0.14.3
v0.14.2
v0.14.1
v0.14.0
v0.13.1
v0.13.0
v0.12.2
v0.12.1
v0.12.0
v0.11.2
v0.11.1
v0.11.0
v0.10.0
v0.9.3
v0.9.2
v0.9.1
v0.9.0
v0.8.2
v0.8.1
v0.8.0
v0.7.6
v0.7.5
v0.7.4
v0.7.3
0.7.2
v.0.7.1
v0.7.0
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.0
Labels
Clear labels
🎨 Design
📖 Docs Update
🐛 Bug
🐛 Bug
:cat2:🐈 Possible duplicate
💿 Database
☕ Open to discussion
💻 Front-End
🐕 Support
🚪 Authentication
🌍 Translations
🔌 API Task
🏭 Back-End
⛲ Upstream
🔨 Feature Request
🛠️ Enhancement
🛠️ Enhancement
🛠️ Enhancement
❤️ Happy feedback
🔒 Security
🔍 Pending Validation
💆 UX
📝 WYSIWYG Editor
🌔 Out of scope
🔩 API Request
:octocat: Admin/Meta
🖌️ View Customization
❓ Question
🚀 Priority
🛡️ Blocked
🚚 Export System
♿ A11y
🔧 Maintenance
> Markdown Editor
pull-request
Mirrored from GitHub Pull Request
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/BookStack#2396
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @floviolleau on GitHub (Sep 18, 2021).
Hi,
Thanks for your amazing project.
I'm using BookStack v21.08.3
I cannot associate a user to a role and I face to a 500 error:
Any ideas?
Thanks a lot
Logs
@ssddanbrown commented on GitHub (Sep 18, 2021):
Hi @floviolleau,
Has anyone attempted editing core app files or made more extensive customizations here?
Within your error, A table prefix of
brain_it attempted to be used which is not something I believe we make configurable by default in any way. Depending on how this has been doing, it could have implications on the foreign references mentioned in the error or it may indicate other changes have occurred.@floviolleau commented on GitHub (Sep 18, 2021):
Hi,
Yes you are exact. I forgot what I did. Because I have only one database, I have to share table names.
So I must use prefixes.
What I did:
Maybe there is something else to do.
Because BookStack is based on Laravel, I think it is small changes and will add a value.
Thanks in advance
@ssddanbrown commented on GitHub (Sep 18, 2021):
Hi @floviolleau,
BookStack does not currently support table prefixes and has not been tested with them so it's possible you might encounter errors.
That said, if configured before migrating the prefix should have been taken into account when the foreign references were created. Did you make the changes after migrating the database (with a manual change of database names to suit) or were the changes made before any database migrations were run?
@floviolleau commented on GitHub (Sep 19, 2021):
Hi,
Thanks for your quick feedback.
You were right. I created the BD and next changed the db prefixes. It solved an issue but not others.
One more thing need to be adapted into the code.
I will propose a PR for this feature
Thanks
@ssddanbrown commented on GitHub (Sep 20, 2021):
Hi @floviolleau,
You may also need to alter any column/table references used in indexes on your instance.
Thanks for offering the PR. I'm somewhat apprehensive to support this officially, since it would really need us to test everything with this feature enabled. It'll also introduce an option that can't really be altered without causing breakages. In addition, it's an extra thing to keep in mind or to enquire about when supporting users. Overall I'm not sure the support effort would be worth the benefit of implementation since this is the first request I can remember for such an option/feature.
Is there a reason you only have a single database to use that has to be shared? Just trying to understand the context in which this feature is required.
@floviolleau commented on GitHub (Sep 20, 2021):
Hi,
Thanks for your message and advices.
I deleted the db and recreated it with DB prefix enabled so all indexes are fine.
My instance of BookStack is running fine without any issues now.
For the feature, I'm on a perso OVH share hosting and I cannot:
This is why I need to prefix all tables to avoid collision with another service.
@ssddanbrown commented on GitHub (Sep 29, 2021):
Thanks @floviolleau.
Now merged for next patch release, I followed up your PR with
025442fcd9. I've taken out the options from the example.envfiles because, as per my last message, I cannot assure a level of testing I'd have happy with for this but I'd still add the feature for those that really need it where are potential sacrifice to stability may be warranted.