[PR #699] [CLOSED] Dont require email confirmation for new users using google auth with … #5694

Closed
opened 2026-02-05 10:14:27 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/699
Author: @thomseddon
Created: 2/7/2018
Status: Closed

Base: releaseHead: feature/automatic-social-reg


📝 Commits (1)

  • c44bf34 Dont require email confirmation for new users using google auth with domain restriction

📊 Changes

1 file changed (+4 additions, -1 deletions)

View changed files

📝 app/Http/Controllers/Auth/RegisterController.php (+4 -1)

📄 Description

…domain restriction

We use G Suite for all our users, we have restricted logins to our own domain only via the "registration restrict" setting and so we don't want to use verification emails.

This is the most basic implementation (which we're now using), please let me know if there is any appetite for this change and if so, if you'd like any extra settings. For example perhaps we could just have an extra setting like "Don't use verification emails with Google login when using domain restriction" ?


🔄 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/699 **Author:** [@thomseddon](https://github.com/thomseddon) **Created:** 2/7/2018 **Status:** ❌ Closed **Base:** `release` ← **Head:** `feature/automatic-social-reg` --- ### 📝 Commits (1) - [`c44bf34`](https://github.com/BookStackApp/BookStack/commit/c44bf342579fbddf705409c8f18a7b9187f7466b) Dont require email confirmation for new users using google auth with domain restriction ### 📊 Changes **1 file changed** (+4 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Http/Controllers/Auth/RegisterController.php` (+4 -1) </details> ### 📄 Description …domain restriction We use G Suite for all our users, we have restricted logins to our own domain only via the "registration restrict" setting and so we don't want to use verification emails. This is the most basic implementation (which we're now using), please let me know if there is any appetite for this change and if so, if you'd like any extra settings. For example perhaps we could just have an extra setting like "Don't use verification emails with Google login when using domain restriction" ? --- <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:14:27 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#5694