[PR #1641] [MERGED] Upgrade laravel to v6 #5856

New Issue

OVERLORD · 2026-02-05T10:18:48+03:00

OVERLORD commented

2026-02-05 10:18:48 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/1641
Author: @ssddanbrown
Created: 9/7/2019
Status: ✅ Merged
Merged: 9/14/2019
Merged by: @ssddanbrown

Base: master ← Head: laravel-upgrade

📝 Commits (7)

213e9d2 Upgraded to Laravel 5.6
6917ea0 Upgraded app to Laravel 5.7
de8a1a3 Updated travis-ci php versions
58f5508 Added mysql service for travis
140298b Updated to Laravel 5.8
cbf9d70 Updated to laravel 6
58a79fc Removed old str_random functions from seeders

📊 Changes

244 files changed (+2561 additions, -1349 deletions)

View changed files

📝 .env.example.complete (+1 -1)
📝 .gitignore (+2 -1)
📝 .travis.yml (+8 -7)
📝 app/Application.php (+6 -1)
📝 app/Auth/Access/SocialAuthService.php (+9 -7)
📝 app/Auth/Access/UserTokenService.php (+3 -2)
📝 app/Auth/Permissions/PermissionsRepo.php (+2 -1)
📝 app/Config/app.php (+5 -4)
📝 app/Config/auth.php (+1 -0)
📝 app/Config/broadcasting.php (+11 -2)
📝 app/Config/cache.php (+1 -1)
📝 app/Config/database.php (+10 -30)
➕ app/Config/hashing.php (+37 -0)
➕ app/Config/logging.php (+82 -0)
📝 app/Config/mail.php (+6 -0)
📝 app/Config/queue.php (+6 -28)
📝 app/Config/services.php (+0 -17)
📝 app/Config/session.php (+6 -1)
📝 app/Entities/BreadcrumbsViewComposer.php (+3 -2)
📝 app/Entities/SearchService.php (+3 -2)

...and 80 more files

📄 Description

WIP pull request to cover #1600.

Notes

Change in PHP version.
Update of password requirements, from 6 to 8 minimum chars.
- Ideally need to update translations to prevent confusion, Only done EN for now.
Possible issue with dotenv parsing changes. Hopefully unlikely in many cases.
Can possibly force MySQL to go over SSL via MYSQL_ATTR_SSL_CA env var. Need to check.

_{🔄 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/1641 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 9/7/2019 **Status:** ✅ Merged **Merged:** 9/14/2019 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `laravel-upgrade` --- ### 📝 Commits (7) - [`213e9d2`](https://github.com/BookStackApp/BookStack/commit/213e9d2941dbedbce8fd639fbf09552ac72ad728) Upgraded to Laravel 5.6 - [`6917ea0`](https://github.com/BookStackApp/BookStack/commit/6917ea088f905ec57e16d8795ad0bcb191c38b52) Upgraded app to Laravel 5.7 - [`de8a1a3`](https://github.com/BookStackApp/BookStack/commit/de8a1a372d9a84358ee1f46a9fe7a0edc7ebba35) Updated travis-ci php versions - [`58f5508`](https://github.com/BookStackApp/BookStack/commit/58f5508b05e9d3265bc79ffc13e1c9b32d031591) Added mysql service for travis - [`140298b`](https://github.com/BookStackApp/BookStack/commit/140298bd96778ded6af37401b83112256ac750bf) Updated to Laravel 5.8 - [`cbf9d70`](https://github.com/BookStackApp/BookStack/commit/cbf9d701af4e510be51fd1f5d2fd8a179534a523) Updated to laravel 6 - [`58a79fc`](https://github.com/BookStackApp/BookStack/commit/58a79fcb19c7a8663c61771a2fe93cde3264c503) Removed old str_random functions from seeders ### 📊 Changes **244 files changed** (+2561 additions, -1349 deletions) <details> <summary>View changed files</summary> 📝 `.env.example.complete` (+1 -1) 📝 `.gitignore` (+2 -1) 📝 `.travis.yml` (+8 -7) 📝 `app/Application.php` (+6 -1) 📝 `app/Auth/Access/SocialAuthService.php` (+9 -7) 📝 `app/Auth/Access/UserTokenService.php` (+3 -2) 📝 `app/Auth/Permissions/PermissionsRepo.php` (+2 -1) 📝 `app/Config/app.php` (+5 -4) 📝 `app/Config/auth.php` (+1 -0) 📝 `app/Config/broadcasting.php` (+11 -2) 📝 `app/Config/cache.php` (+1 -1) 📝 `app/Config/database.php` (+10 -30) ➕ `app/Config/hashing.php` (+37 -0) ➕ `app/Config/logging.php` (+82 -0) 📝 `app/Config/mail.php` (+6 -0) 📝 `app/Config/queue.php` (+6 -28) 📝 `app/Config/services.php` (+0 -17) 📝 `app/Config/session.php` (+6 -1) 📝 `app/Entities/BreadcrumbsViewComposer.php` (+3 -2) 📝 `app/Entities/SearchService.php` (+3 -2) _...and 80 more files_ </details> ### 📄 Description WIP pull request to cover #1600. ## Notes * Change in PHP version. * Update of password requirements, from 6 to 8 minimum chars. * Ideally need to update translations to prevent confusion, Only done EN for now. * Possible issue with [dotenv parsing changes](https://laravel.com/docs/5.8/upgrade#environment-variable-parsing). Hopefully unlikely in many cases. * Can possibly force MySQL to go over SSL via `MYSQL_ATTR_SSL_CA` env var. Need to check. --- <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:18:48 +03:00

OVERLORD closed this issue

2026-02-05 10:18:49 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#5856