-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
fix: Panel Installer #951
fix: Panel Installer #951
Conversation
…directing users to /install to initiate the installation process.
Also fixes minor issues like spelling / sentence mistakes. Refactored some variable names, and made it actually show error message on the page instead of only in logs. Now email settings have select input for mail method and encryption.
Fix installer setting pterodactyl settings without encryption, causing 500 Error in settings page.
I'd like to take a look and implement "Post-Installation Issues with Redis Configuration" I assume it will be an extra step after/before database configuration and preferably with a verification that the redis server connection was successful? |
@jameskitt616 yep exactly |
I think the best would be after the main database its just some env value to set and test |
Update Panel Installer add redis configuration add timezone configuration fix and clean code
LGTM, couldnt test it for sure. Please fix conflicts too. |
idk how to comment on specific fixes but checking this out I found a couple things: during install I still got this ` at vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php:79 i Your app key is missing: |
@S0ly Should the installer handle the app key generation too? |
I suppose that if he can he should no ? like if no app key is present maybe generate one ? since he do all the rest, but if you think it should not be handled by the installer we could have the script of weez doing it, since we now have an installation script |
Wasn't that fixed somewhere? |
its why I called it " Inconsistent Behavior" sometimes it happend sometimes not |
Most likely file/directory permissions were missing for the user that runs php -> thus cannot create a app key? |
Not really, if it was a permission issue, it would need to fail everytime. |
I (think) i fixed this issue with my timezone fix with the new Installer |
I found out if there is a fresh install with no |
I invite contributions to address several issues with the installer for the ctrlpanel:
Issues to be Fixed:
Unconfigured Panel Handling: Currently, the ctrlpanel displays an error when initiated without an install.lock file. It could be enhanced by implementing a mechanism to detect the absence of install.lock and redirecting users to /install to initiate the installation process.
Deprecated PHP Function Error in trustedproxy.php: The consistent appearance of the error message "Deprecated: explode(): Passing null to parameter ($string) of type string is deprecated in /var/www/html/config/trustedproxy.php on line 29" necessitates immediate resolution to prevent deprecated function calls.
Docker MySQL and Unnecessary Checks: The Docker setup lacks MySQL, which may be unnecessary for some users as they store their databases outside the server. Additionally, HTTPS checks are redundant as modern web browsers already provide warnings.
Failed PHP Extension Check for Redis: Despite the potential necessity of Redis, the PHP extension check fails to verify its presence. This issue requires rectification to ensure proper functionality.
Inconsistent Behavior with php artisan migrate --seed --force: Occasional failures of this command, accompanied by deprecated errors and timezone-related notices, demand investigation to ensure seamless execution.
Post-Installation Issues with Redis Configuration: Following installation, the ctrlpanel encounters difficulties connecting to Redis, necessitating manual configuration in the .env file.
Unencrypted Tokens in Settings Table: The installer sets tokens directly in the settings table without encryption, causing errors when Laravel attempts to decrypt them. This requires modification to ensure tokens are encrypted properly during installation.
Contributors are encouraged to address these issues to enhance the reliability and usability of the ctrlpanel installer. Thank you for your contributions!