Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🚀 Feature: Add warning to prevent lockout admin user when disabling normal login #630

Open
olsonnn opened this issue Oct 2, 2024 · 1 comment
Labels
feature New feature or request

Comments

@olsonnn
Copy link

olsonnn commented Oct 2, 2024

🔖 Feature description

Happens if you do this:

  1. Configure social login (Microsoft)
  2. Test social login
  3. Disable normal user /password login.
  4. No admin user anymore as this was not done in step 2

🎤 Pitch

How to prevent
During disable normal login, check if there at least ONE admin in social login.
If not, do not allow to disable it/ show warning

@olsonnn olsonnn added the feature New feature or request label Oct 2, 2024
@InvitedToHell
Copy link
Contributor

Since I crashed into the same issue, I want to provide a workaround until a warning will be implemented.

  1. Open a terminal at the location of your docker-compose.yml
  2. Install sqlite3 (debian/ubuntu: sudo apt install sqlite3)
  3. Run sudo sqlite3 data/pingvin-share.db
  4. Run UPDATE Config SET value = 'false' WHERE name = 'oauth.disablePassword';
  5. CTRL + C to exit and then restart Pingvin Share with a quick docker compose down followed by docker compose up -d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants