starred/panel-pelican-dev
1
0
Fork 0
mirror of https://github.com/pelican-dev/panel.git synced 2026-05-04 18:00:48 +03:00
Code Issues 41 Packages Projects Releases 10 Wiki Activity

[PR #903] [MERGED] OAuth improvements #1047

New Issue
Closed
opened 2026-02-05 18:02:14 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-05 18:02:14 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/903
Author: @Boy132
Created: 1/10/2025
Status: ✅ Merged
Merged: 1/15/2025
Merged by: @Boy132

Base: main ← Head: boy132/oauth-settings

📝 Commits (10+)

  • 6cb9844 rework oauth provider creation & lodaing
  • 0f3d93a add separate setup form
  • 2803d02 use wizard for setup
  • 33ee4e3 add provider class for discord
  • 3a6af89 cleanup and fixes
  • ed82f3c don't throw exception when creating duplicate provider
  • 8751f45 update profile and login pages
  • 0ae63ad did not mean to remove the whole else, oops
  • 1c2ee31 use import
  • 2b431da Merge remote-tracking branch 'upstream/main' into boy132/oauth-settings

📊 Changes

12 files changed (+504 additions, -212 deletions)

View changed files

➕ app/Extensions/OAuth/Providers/AuthentikProvider.php (+65 -0)
➕ app/Extensions/OAuth/Providers/CommonProvider.php (+36 -0)
➕ app/Extensions/OAuth/Providers/DiscordProvider.php (+57 -0)
➕ app/Extensions/OAuth/Providers/GithubProvider.php (+56 -0)
➕ app/Extensions/OAuth/Providers/OAuthProvider.php (+112 -0)
➕ app/Extensions/OAuth/Providers/SteamProvider.php (+74 -0)
📝 app/Filament/Admin/Pages/Settings.php (+50 -58)
📝 app/Filament/Pages/Auth/EditProfile.php (+20 -14)
📝 app/Filament/Pages/Auth/Login.php (+12 -8)
📝 app/Http/Controllers/Auth/OAuthController.php (+3 -2)
📝 app/Providers/AppServiceProvider.php (+19 -16)
📝 config/auth.php (+0 -114)

📄 Description

Removes the oauth config stuff from auth.php and makes use of classes.
This improves the creation, loading and setup of oauth providers.

Example for the new setup wizard:
image

Note: Not all providers have "advanced" setup wizards yet!

🔄 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/903 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 1/10/2025 **Status:** ✅ Merged **Merged:** 1/15/2025 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `boy132/oauth-settings` --- ### 📝 Commits (10+) - [`6cb9844`](https://github.com/pelican-dev/panel/commit/6cb9844182e9c43df69127c80244c2961a80591a) rework oauth provider creation & lodaing - [`0f3d93a`](https://github.com/pelican-dev/panel/commit/0f3d93a13b86ee99d812978140e60e6575ad9cac) add separate setup form - [`2803d02`](https://github.com/pelican-dev/panel/commit/2803d02e38f75ed453dc1503fa03a37205e198f7) use wizard for setup - [`33ee4e3`](https://github.com/pelican-dev/panel/commit/33ee4e355b6f2bf38cfcb1c084584a31f8c480f9) add provider class for discord - [`3a6af89`](https://github.com/pelican-dev/panel/commit/3a6af89360a5b14d8dbb01510b717f44c63cb136) cleanup and fixes - [`ed82f3c`](https://github.com/pelican-dev/panel/commit/ed82f3cbc2e7d5adde4525d51d32d8ec4496ebf7) don't throw exception when creating duplicate provider - [`8751f45`](https://github.com/pelican-dev/panel/commit/8751f45f61976cf89ccfd6306528770c2fba1aaf) update profile and login pages - [`0ae63ad`](https://github.com/pelican-dev/panel/commit/0ae63ad73e280ce342c2eb7fb3c6f559768b18dd) did not mean to remove the whole else, oops - [`1c2ee31`](https://github.com/pelican-dev/panel/commit/1c2ee31af26855c6ef812dd7afe42acfd6d16ed8) use import - [`2b431da`](https://github.com/pelican-dev/panel/commit/2b431dadcd826a4c4e4a0712a74dff88f5a972ff) Merge remote-tracking branch 'upstream/main' into boy132/oauth-settings ### 📊 Changes **12 files changed** (+504 additions, -212 deletions) <details> <summary>View changed files</summary> ➕ `app/Extensions/OAuth/Providers/AuthentikProvider.php` (+65 -0) ➕ `app/Extensions/OAuth/Providers/CommonProvider.php` (+36 -0) ➕ `app/Extensions/OAuth/Providers/DiscordProvider.php` (+57 -0) ➕ `app/Extensions/OAuth/Providers/GithubProvider.php` (+56 -0) ➕ `app/Extensions/OAuth/Providers/OAuthProvider.php` (+112 -0) ➕ `app/Extensions/OAuth/Providers/SteamProvider.php` (+74 -0) 📝 `app/Filament/Admin/Pages/Settings.php` (+50 -58) 📝 `app/Filament/Pages/Auth/EditProfile.php` (+20 -14) 📝 `app/Filament/Pages/Auth/Login.php` (+12 -8) 📝 `app/Http/Controllers/Auth/OAuthController.php` (+3 -2) 📝 `app/Providers/AppServiceProvider.php` (+19 -16) 📝 `config/auth.php` (+0 -114) </details> ### 📄 Description Removes the oauth config stuff from `auth.php` and makes use of classes. This improves the creation, loading and setup of oauth providers. Example for the new setup wizard: ![image](https://github.com/user-attachments/assets/8570cdfc-a6b5-49f0-88cb-96503035dabd) _Note: Not all providers have "advanced" setup wizards yet!_ --- <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 18:02:14 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
not confirmed
pull-request
Mirrored from GitHub Pull Request
 👑 sponsor
💰 fund
🔴 hard
🟡 medium
🟡 medium
🟡 medium
🟢 easy
🟢 easy
🟢 easy
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#1047
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?