Skip to content

[Merge] 3.x into 4.x (May 26th 2026)#1134

Merged
RichardAnderson merged 22 commits into
vitodeploy:4.xfrom
RichardAnderson:rebase/3_to_4/260526
May 27, 2026
Merged

[Merge] 3.x into 4.x (May 26th 2026)#1134
RichardAnderson merged 22 commits into
vitodeploy:4.xfrom
RichardAnderson:rebase/3_to_4/260526

Conversation

@RichardAnderson
Copy link
Copy Markdown
Member

Merges 3.x changes into 4.x.

Ensure merge commits and not squash to preserve mergeability support for any future 3.x updates.

saeedvaziry and others added 22 commits March 3, 2026 18:36
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* throw error instead of suppressing

* Add error message from generic error

* fixes

* fix naming issue

align with interface
* dns improvements

* copilot fixes
…g multi-host accounts (vitodeploy#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.
Co-authored-by: Saeed Vaziry <61919774+saeedvaziry@users.noreply.github.com>
@RichardAnderson RichardAnderson marked this pull request as ready for review May 26, 2026 16:39
@saeedvaziry
Copy link
Copy Markdown
Member

saeedvaziry commented May 26, 2026

We might need run npm audits and composer updates again on 4.x

@RichardAnderson RichardAnderson merged commit fc950d6 into vitodeploy:4.x May 27, 2026
3 checks passed
@RichardAnderson RichardAnderson deleted the rebase/3_to_4/260526 branch May 27, 2026 08:58
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.

3 participants