Skip to content

Require Classic Warden locale hardening catalogue - #482

Merged
MadMaxMangos merged 1 commit into
masterfrom
feature/classic-warden-locale-hardening
Aug 25, 2026
Merged

Require Classic Warden locale hardening catalogue#482
MadMaxMangos merged 1 commit into
masterfrom
feature/classic-warden-locale-hardening

Conversation

@MadMaxMangos

@MadMaxMangos MadMaxMangos commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • require World DB 22/06/004 Warden_Locale_Hardening
  • expand the verified Warden catalogue fixtures from three to eight Classic Windows locale profiles
  • cover the nine-check initial batch and recurring/aggressive scheduling for the two new interface-signature dispatch invariants
  • pin the exact 5875/6005/6141 request and response vectors without changing the existing Warden wire protocol

Validation

  • RelWithDebInfo build and INSTALL completed with Eluna and PlayerBots enabled
  • CTest: 16/16 targets passed
  • mangos_tests: 354/354 passed
  • Warden database tests: 27/27 passed
  • Observe-mode live acceptance passed on clean builds 5875, 6005, and 6141
  • the intentional modified zhTW executable produced confirmed mismatches only for checks 65537 and 65538, with no kick, ban, incident, or protocol failure

Dependency

Requires mangoszero/database#159. Merge the database PR first, then this server PR.


This change is Reviewable

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@AppVeyorBot

Copy link
Copy Markdown

@MadMaxMangos
MadMaxMangos merged commit 2977c96 into master Aug 25, 2026
8 checks passed
@MadMaxMangos
MadMaxMangos deleted the feature/classic-warden-locale-hardening branch August 25, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants