[PR #1676] [MERGED] Replace str_random with Str::random #1520

Closed
opened 2026-02-04 19:53:42 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1676
Author: @notAreYouScared
Created: 9/6/2025
Status: Merged
Merged: 9/6/2025
Merged by: @notAreYouScared

Base: mainHead: rabbit/replace-depre


📝 Commits (2)

  • e5e5c80 replace str_random with Str::random
  • 9efc8d5 Revert public removal

📊 Changes

11 files changed (+22 additions, -12 deletions)

View changed files

📝 app/Filament/Admin/Resources/ApiKeyResource/Pages/CreateApiKey.php (+2 -1)
📝 app/Filament/Server/Resources/DatabaseResource/Pages/ListDatabases.php (+2 -1)
📝 app/Helpers/Utilities.php (+2 -1)
📝 app/Services/Api/KeyCreationService.php (+2 -1)
📝 app/Services/Databases/DatabaseManagementService.php (+2 -1)
📝 app/Services/Users/ToggleTwoFactorService.php (+2 -1)
📝 app/Services/Users/UserCreationService.php (+1 -1)
📝 database/migrations/2017_09_23_173628_RemoveDaemonSecretFromServersTable.php (+2 -1)
📝 database/migrations/2017_09_23_185022_RemoveDaemonSecretFromSubusersTable.php (+2 -1)
📝 database/migrations/2017_10_02_202007_ChangeToABetterUniqueServiceConfiguration.php (+2 -1)
📝 database/migrations/2017_11_19_122708_MigratePubPrivFormatToSingleKey.php (+3 -2)

📄 Description

rabbit suggested due to str_random being deprecated


🔄 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/pelican-dev/panel/pull/1676 **Author:** [@notAreYouScared](https://github.com/notAreYouScared) **Created:** 9/6/2025 **Status:** ✅ Merged **Merged:** 9/6/2025 **Merged by:** [@notAreYouScared](https://github.com/notAreYouScared) **Base:** `main` ← **Head:** `rabbit/replace-depre` --- ### 📝 Commits (2) - [`e5e5c80`](https://github.com/pelican-dev/panel/commit/e5e5c8062138b7177099b84aa62a557cd795c46f) replace `str_random` with `Str::random` - [`9efc8d5`](https://github.com/pelican-dev/panel/commit/9efc8d510ad67b7825976af6cb595beb7268bfe3) Revert public removal ### 📊 Changes **11 files changed** (+22 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `app/Filament/Admin/Resources/ApiKeyResource/Pages/CreateApiKey.php` (+2 -1) 📝 `app/Filament/Server/Resources/DatabaseResource/Pages/ListDatabases.php` (+2 -1) 📝 `app/Helpers/Utilities.php` (+2 -1) 📝 `app/Services/Api/KeyCreationService.php` (+2 -1) 📝 `app/Services/Databases/DatabaseManagementService.php` (+2 -1) 📝 `app/Services/Users/ToggleTwoFactorService.php` (+2 -1) 📝 `app/Services/Users/UserCreationService.php` (+1 -1) 📝 `database/migrations/2017_09_23_173628_RemoveDaemonSecretFromServersTable.php` (+2 -1) 📝 `database/migrations/2017_09_23_185022_RemoveDaemonSecretFromSubusersTable.php` (+2 -1) 📝 `database/migrations/2017_10_02_202007_ChangeToABetterUniqueServiceConfiguration.php` (+2 -1) 📝 `database/migrations/2017_11_19_122708_MigratePubPrivFormatToSingleKey.php` (+3 -2) </details> ### 📄 Description rabbit suggested due to str_random being deprecated --- <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-04 19:53:42 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1520