Skip to content

Fixed NoAuth option #5737

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

Closed
wants to merge 1 commit into from
Closed

Fixed NoAuth option #5737

wants to merge 1 commit into from

Conversation

AJolly
Copy link

@AJolly AJolly commented Jul 5, 2025

✍️ Description

the previous method created a user ID with a password of 1 which did not meet complexity requirements, this consolidates the config initialization and fixes the no-auth option.

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.
    Has the same previous hard coded secrets.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

the previous method created a user ID with a password of 1 which did not meet complexity requirements
@AJolly AJolly requested a review from a team as a code owner July 5, 2025 19:41
@MickLesk
Copy link
Member

MickLesk commented Jul 7, 2025

Doesnt work.

image

/usr/local/community-scripts/filebrowser.db does not exist. Please run 'filebrowser config init' first.

You have removed important commands, which means that the database is not initialized correctly. I built the commands this way because it worked without any problems.

@MickLesk MickLesk closed this Jul 9, 2025
@AJolly
Copy link
Author

AJolly commented Jul 9, 2025

huh. I'll go retest. Creating the user with the password of 1 did not work (when i tried the command directly), so no-auth wouldnt work for me.
Also very weird - the setup script means filebrowser conf show doesnt work.

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

Successfully merging this pull request may close these issues.

2 participants