Fix workflows throwing error 500#1137
Closed
saeedvaziry wants to merge 113 commits into
Closed
Conversation
Added set -e and set -o pipefail to scripts executed in runScript method. This ensures that when commands like migration or git pull fail, the script stops execution and subsequent commands are not executed. Previously, scripts would continue even when a command failed, which could lead to incorrect results and inconsistent system states. Co-authored-by: Saeed Vaziry <61919774+saeedvaziry@users.noreply.github.com>
* Add NodeJS website using mise supporting node versions per site * fix mise * wip * fix worker
* Optional cron job naming added * Fix: CronJob model, test and form formatting
* New environment varialbe editor experience * add paste from clipboard * AI fixes * fixes
* Migrate Vito to Octane with FrankenPHP * changes
This reverts commit 9fa1468.
* WIP * precommit updates * copilot feedback
* wip * fix issues with tests * pre-commit fixes * copilot fixes
* wip * pr review * pre-commit fixes * added bun for laravel types * refactor * copilot fixes * refactor GetIsolatedUsers
* WIP - store new values * monitoring ui * overview page + server table * pre-commit * calculate percentage * refactor * copilot feedback * add timeout
* check for updates automatically * resolve copilot feedback
* vito splash * refactor for caddy support * copilot feedback * more feedback
* service logs * review fixes * copilot feedback + fixes * moved ServiceLog to DTOs * fixes * fix dodgy merge
* wip * various fixes * hide site banners on install * pint * removed dead weight
* WIP * refactor of site install * pre-commit * Correct order * worker * refactor - tooling picker+selector * wip * wip * improved deployment ui * resolve issue with alert for tooling * isolated_user model + migration * broadcast events fixes * worker protection & app card * Refactor site types * claude review * full review * missed & refactored tests * copilot fixes * misc fixes * reverted a fix for this PR
#1104) Custom::create rejected a server whenever `id -u vito` succeeded, which came from #942 against the 3.x bare-metal installer (scripts/install.sh:50). In 4.x Vito ships as a Docker container and does not write a `vito` system user to the host, so the old check false-positives on any unrelated host that already has a `vito` Linux user. Combining the user check with `test -f /home/vito/vito/artisan` (the clone path at scripts/install.sh:197) keeps the original root-only bar and removes the false positive.
* required tooling * fix for all site types
* available tooling * strip comment
* Build version 3.20.2 * Fix missing authorization in workflow site creation (#1036) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * npm audit * Build version 3.20.3 * Better error handling and exception cases for Domains (#1049) * throw error instead of suppressing * Add error message from generic error * fixes * fix naming issue align with interface * DNS Management Improvements (#1050) * dns improvements * copilot fixes * [Domains] Edit DNS Provider (#1051) * edit-dns-provider * fixes * Build version 3.21.0 * [Fix] Resolve SSH Directory Issue (#1054) * cd to home directory * fixes * [Fix] Resolves Domain Dialog Issues (#1055) * fixes issues * Updated tests * [FIX] Improve .env update validation (#1061) * npm audit * Build version 3.21.1 * update packages * Build version 3.21.2 * [FIX] Match synced MySQL users by (username, host) to avoid collapsing multi-host accounts (#1075) SyncDatabaseUsers previously matched existing database_users rows by username only. MySQL and MariaDB treat 'user'@'localhost' and 'user'@'127.0.0.1' as independent accounts with potentially different passwords and grants, so collapsing them onto a single row caused silent data corruption: the databases column was overwritten in an order-dependent way, additional host variants were never inserted, and the stored host no longer matched the grants shown under it. The lookup now scopes by (username, host) whenever the handler returns a non-empty host, aligning sync with CreateDatabaseUser, UpdateDatabaseUser and DeleteDatabaseUser. PostgreSQL keeps the original username-only match because its get-users-list view returns an empty host (roles are host-agnostic); an unconditional composite filter would miss the existing UI-created row and insert a duplicate (rolname, '') record on every sync. Adds regression tests covering MySQL multi-host sync, MySQL sync idempotency and the PostgreSQL no-duplicate-row invariant. * Audit third-party packages (#1077) * Build version 3.21.3 * [FIX] Use server SSH port for default Firewall Rules (#1086) * [Fix] Resolve Dropbox backup issue (#1099) Co-authored-by: Saeed Vaziry <61919774+saeedvaziry@users.noreply.github.com> * Build version 3.22.0 --------- Co-authored-by: saeedvaziry <61919774+saeedvaziry@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Saeed Vaziry <mr.saeedvaziry@gmail.com> Co-authored-by: Erhan ÜRGÜN <405591@ogr.ktu.edu.tr>
…missing webhook gracefully The Discord notification now uses rich embeds for better visibility and trims PR links from release notes to reduce noise. Added explicit continue-on-error so a missing or failing webhook doesn't block the release. The step also now sanitizes long release notes (>3800 chars) and differentiates stable vs pre-release channels by color and footer. Output the notes from fetch-release instead of cleaned-notes (which no longer exists after consolidating the notification logic).
The release workflow now builds and pushes Docker images for both amd64 and arm64 platforms. A new docker-tags job determines which tags to apply based on the release branch and prerelease status: version-specific tags always, major version tags (1.x/2.x/3.x/4.x) for all releases, and latest for stable releases on 3.x and 4.x branches only. Separate build jobs run on their respective architectures to avoid cross-compilation overhead and ensure native builds for each platform.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.