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

[Enhancement] Improved ITFlow installation script with logging, security, and user experience updates #7

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

SyNode-IT
Copy link

Hello,

I've made some updates to the ITFlow installation script that I believe could improve its functionality and user experience. Here are the key changes:

  1. Logging system: Added detailed logging to /var/log/itflow_install.log for easier troubleshooting.

  2. MariaDB setup improvements:

    • Automated mysql_secure_installation using expect
    • Added creation of a MariaDB root password
    • Improved error checking for MariaDB installation and configuration
  3. Enhanced error handling:

    • Added checks for required binaries
    • Improved error messages and exit conditions
  4. User interface enhancements:

    • Added numbered steps with progress messages
    • Extended use of color-coded messages for better readability
  5. Package management:

    • Added 'expect' to the list of installed packages
  6. Code structure:

    • Reorganized into more clearly defined functions
    • Improved comments and overall readability
  7. Additional features:

    • Added display of MariaDB root password at the end of installation
    • Included log file location in the final message

These changes aim to make the installation process more robust and user-friendly. I've tested the script successfully on Debian 12 and Ubuntu 22.04.

Do you think these improvements align with your vision for the project? I'd be happy to discuss any aspects of these changes or make adjustments if needed.

Looking forward to your feedback.

Best regards,

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

Successfully merging this pull request may close these issues.

1 participant