Disable ASAN CI job until Linux widechar paths are fixed

Per #289 discussion with @irodushka: rather than keep ASAN running
under continue-on-error, disable it outright until the Unicode rewrite
lands (draft PR #291, tracked as #287 Tier 1b). Re-enable the matrix
entry once #291 merges.

Valgrind remains strict.

Author: fdcastel

.github/workflows/build-and-test.yml

@@ -56,21 +56,19 @@ jobs:
             firebird-branch: master
 
           # ── Linux x64 sanitizers (Debug, Firebird 5.0.3) ─────────────────────
-          - os: ubuntu-22.04
-            sanitizer: Asan
-            firebird-version: '5.0.3'
+          # ASAN is temporarily disabled — re-enable once the Linux widechar
+          # conversion paths in MainUnicode.cpp are rewritten (#287 Tier 1b,
+          # draft PR #291). Until then it fails on a heap-buffer-overflow in
+          # SQLGetDiagRecW.
+          # - os: ubuntu-22.04
+          #   sanitizer: Asan
+          #   firebird-version: '5.0.3'
           - os: ubuntu-22.04
             sanitizer: Valgrind
             firebird-version: '5.0.3'
 
     runs-on: ${{ matrix.os }}
 
-    # Temporary: the ASAN job is expected to fail on the heap-buffer-overflow
-    # inside SQLGetDiagRecW until the Linux widechar conversion paths in
-    # MainUnicode.cpp are rewritten. Tracked in issue #287 Tier 1b. Revert
-    # this back to strict once the Unicode fix PR lands.
-    continue-on-error: ${{ matrix.sanitizer == 'Asan' }}
-
     steps:
       - uses: actions/checkout@v6
         with: