From f72054f61f5f191179de94e409791dedc4e44c97 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=90=D0=BD=D1=82=D0=BE=D0=BD=20=D0=A2=D1=80=D0=BE=D1=88?= =?UTF-8?q?=D0=B8=D0=BD?= Date: Tue, 22 Sep 2026 22:26:20 +0300 Subject: [PATCH] ui: stop showing the all-FF record marker as a field; label it in the diff (v3.1.4) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit A profile check comparing an empty PNN record against a populated one showed a cryptic 'Empty: yes' row: the decoders return {empty: true} for an all-FF record and efFlatten() rendered that marker as a pseudo-field ('empty' prettified to 'Empty', the boolean as 'yes'). - efFlatten() skips the marker (the decoded views already show their own translated 'empty' note next to the record). - efDiffData() fills the cells of a side that decodes to an all-FF record with 'Empty (all FF)' (RU: 'Пусто (только FF)') instead of leaving them blank; a field merely missing from a non-empty record stays blank. - tests: efFlatten marker, one-sided and both-sided empty diffs, and the PNN case from the report; help EN/RU note the new text. - sw.js simple-v239. --- frontend/help-ru.html | 2 +- frontend/help.html | 2 +- frontend/index.html | 17 +++++++++++++++-- frontend/sw.js | 2 +- frontend/tests/ef_decode.test.js | 24 ++++++++++++++++++++++++ pyproject.toml | 2 +- pysim_simple_server/server.py | 2 +- 7 files changed, 44 insertions(+), 7 deletions(-) diff --git a/frontend/help-ru.html b/frontend/help-ru.html index 089e818..9b0dd90 100644 --- a/frontend/help-ru.html +++ b/frontend/help-ru.html @@ -421,7 +421,7 @@
  • Атрибуты файла — тип файла, размер, длина записи и число записей из FCP-шаблона (любой можно оставить незаданным).
  • Проверить содержимое (опционально) — Точное (точное совпадение hex) или Маска, где ? — шаблон на один полубайт (маска без ? — совпадение префикса, например 0891 для MCC/MNC из IMSI). Для record-файлов хранится список по записям.
    • -

    Проверить карту выполняет каждое правило на подключённой карте и показывает строку прогресса и отчёт прохождения. Рядом с путём файла указывается, что именно проверялось (например, тип файла и размер, содержимое или полный FCI); если часть проверок прошла, а часть нет — каждый аспект помечается (тип файла ✓, размер ✗, содержимое ✓), а расхождения расписываются ниже. Несовпавшие сырые данные (FCI, содержимое, данные записей) показываются как поля только для чтения с моноширинным шрифтом — ожидаемое над фактическим, в одной и той же колонке — для удобного сравнения; расхождения содержимого дополнительно показывают для файлов с декодером декодированное сравнение по полям (только отличающиеся поля, с заголовками колонок «ожидаемое/фактическое» над значениями; если декодированные значения совпадают, вместо таблицы выводится пометка об этом), а для расхождений FCI дополнительно показывается декодированное сравнение по параметрам (размер файла, дескриптор/структура, жизненный цикл, FID, SFI, проприетарные параметры…). Декодированный просмотр FCI также отображается рядом с полем FCI hex при редактировании правила. Если данные FCI повреждены, показывается всё, что удалось декодировать до места ошибки, вместе с явным сообщением об ошибке декодирования. Для record-файлов при расхождении содержимого добавляется пометка совпадающие записи: 1-5, 7-10 со списком записей, которые совпали. В отчёте поля расхождений и колонки сравнения FCI подписаны ожидалось (имя профиля) и фактически (ICCID карты), а в заголовке отчёта выводится Результаты проверки профиля: <профиль> → <ICCID карты>. Опция «Только расхождения» в заголовке отчёта скрывает все совпавшие файлы и оставляет только несовпадения и ошибки.

    +

    Проверить карту выполняет каждое правило на подключённой карте и показывает строку прогресса и отчёт прохождения. Рядом с путём файла указывается, что именно проверялось (например, тип файла и размер, содержимое или полный FCI); если часть проверок прошла, а часть нет — каждый аспект помечается (тип файла ✓, размер ✗, содержимое ✓), а расхождения расписываются ниже. Несовпавшие сырые данные (FCI, содержимое, данные записей) показываются как поля только для чтения с моноширинным шрифтом — ожидаемое над фактическим, в одной и той же колонке — для удобного сравнения; расхождения содержимого дополнительно показывают для файлов с декодером декодированное сравнение по полям (только отличающиеся поля, с заголовками колонок «ожидаемое/фактическое» над значениями; если декодированные значения совпадают, вместо таблицы выводится пометка об этом, а сторона, чья запись целиком из FF, показывает в своих ячейках Пусто (только FF)), а для расхождений FCI дополнительно показывается декодированное сравнение по параметрам (размер файла, дескриптор/структура, жизненный цикл, FID, SFI, проприетарные параметры…). Декодированный просмотр FCI также отображается рядом с полем FCI hex при редактировании правила. Если данные FCI повреждены, показывается всё, что удалось декодировать до места ошибки, вместе с явным сообщением об ошибке декодирования. Для record-файлов при расхождении содержимого добавляется пометка совпадающие записи: 1-5, 7-10 со списком записей, которые совпали. В отчёте поля расхождений и колонки сравнения FCI подписаны ожидалось (имя профиля) и фактически (ICCID карты), а в заголовке отчёта выводится Результаты проверки профиля: <профиль> → <ICCID карты>. Опция «Только расхождения» в заголовке отчёта скрывает все совпавшие файлы и оставляет только несовпадения и ошибки.

    Опции сканирования «Профиль с карты»

    Диалог сканирования запрашивает имя профиля и предлагает селектор «Проверка FCP/FCI» (те же три режима, по умолчанию Тип файла + размер (FCP)), применяемый ко всем создаваемым правилам, а также список «Игнорировать содержимое файлов» (все отмечены по умолчанию, кроме EF.ARR; флажок в заголовке отмечает или снимает весь список) часто перезаписываемых файлов, содержимое которых пропускается: EF.LOCI, EF.PSLOCI, EF.EPSLOCI, EF.5GS3GPPLOCI, EF.Keys, EF.KeysPS, EF.SMS, EF.Kc, EF.KcGPRS, EF.LOCIGPRS, EF.CBMID, EF.SMSS, EF.ACC, EF.EPSNSC, EF.START-HFN, EF.ARR. Ещё две отмеченные по умолчанию опции «Сравнивать первые 4 байта для» EF.IMSI и EF.ICCID захватывают содержимое этих файлов как маску только первых 4 байт (снимите для точного сравнения). Строка прогресса показывает N / всего файлов с текущим путём файла во время сканирования; при сканировании опции скрываются, а кнопки блокируются. Правила создаются только для файлов, которые реально существуют на карте (возвращён FCP-шаблон); отсутствующие файлы пропускаются. Пользовательские файлы из подвкладки «Пользовательские файлы» включаются с той же проверкой существования. Тот же диалог и опции использует «Профиль из снимка»: вместо карты обходятся захваченные файлы выбранного снимка; для файлов, содержимое которых не было захвачено, правило создаётся без проверки содержимого (при последующей проверке профиля они помечаются как непроверяемые).

    diff --git a/frontend/help.html b/frontend/help.html index 46b53ab..3c68337 100644 --- a/frontend/help.html +++ b/frontend/help.html @@ -421,7 +421,7 @@
  • File attributes — file type, size, record length and record count, taken from the FCP template (any may be left unset).
  • Check contents (optional) — Exact hex equality, or Mask where ? is a per-nibble wildcard (a mask with no ? is a prefix match, e.g. 0891 for the IMSI MCC/MNC). Record files store a per-record list.
    • -

    Check card runs every rule against the equipped card and shows a live progress line plus a pass/fail report. Each row states exactly what was verified next to the file path (e.g. filetype and size, contents or exact FCI); when some checks pass and others fail, each aspect is marked (filetype ✓, size ✗, contents ✓) with the mismatches detailed below. Mismatched raw data (FCI, contents, record data) is shown as read-only monospace fields — expected above actual, aligned in the same column — for easy comparison; content mismatches additionally show, for files with a decoder, a decoded per-field comparison (only the differing fields, with expected/actual column headers above the side-by-side values; when the decoded values match, a note says so instead of the table), and FCI mismatches additionally show a decoded per-parameter comparison (file size, file descriptor/structure, life cycle, FID, SFI, proprietary parameters…). A decoded FCI preview is also shown beside the FCI hex field while editing a rule. If the FCI data is corrupt, whatever was decoded before the faulty part is shown together with an explicit decode-failure note. For record files with a contents mismatch, a matching records: 1-5, 7-10 note lists the records that did match. In the report the mismatch fields and FCI comparison columns are labelled expected (profile name) and actual (card ICCID), and the results header reads Profile verification results for: <profile> → <card ICCID>. An Only mismatches option in the results header hides all passing files and keeps only failures and errors.

    +

    Check card runs every rule against the equipped card and shows a live progress line plus a pass/fail report. Each row states exactly what was verified next to the file path (e.g. filetype and size, contents or exact FCI); when some checks pass and others fail, each aspect is marked (filetype ✓, size ✗, contents ✓) with the mismatches detailed below. Mismatched raw data (FCI, contents, record data) is shown as read-only monospace fields — expected above actual, aligned in the same column — for easy comparison; content mismatches additionally show, for files with a decoder, a decoded per-field comparison (only the differing fields, with expected/actual column headers above the side-by-side values; when the decoded values match, a note says so instead of the table, and a side whose record is all-FF shows Empty (all FF) in its cells), and FCI mismatches additionally show a decoded per-parameter comparison (file size, file descriptor/structure, life cycle, FID, SFI, proprietary parameters…). A decoded FCI preview is also shown beside the FCI hex field while editing a rule. If the FCI data is corrupt, whatever was decoded before the faulty part is shown together with an explicit decode-failure note. For record files with a contents mismatch, a matching records: 1-5, 7-10 note lists the records that did match. In the report the mismatch fields and FCI comparison columns are labelled expected (profile name) and actual (card ICCID), and the results header reads Profile verification results for: <profile> → <card ICCID>. An Only mismatches option in the results header hides all passing files and keeps only failures and errors.

    “Profile from card” scan options

    The scan dialog asks for a profile name and offers a “FCP/FCI check” selector (the same three modes above, default Filetype + size) applied to every generated rule, plus an “Ignore contents of files” checklist (all checked by default except EF.ARR; the header checkbox checks or unchecks the whole list) of frequently-overwritten files whose contents are skipped: EF.LOCI, EF.PSLOCI, EF.EPSLOCI, EF.5GS3GPPLOCI, EF.Keys, EF.KeysPS, EF.SMS, EF.Kc, EF.KcGPRS, EF.LOCIGPRS, EF.CBMID, EF.SMSS, EF.ACC, EF.EPSNSC, EF.START-HFN, EF.ARR. Two further checked-by-default options “Match first 4 bytes for” EF.IMSI and EF.ICCID capture those files’ contents as a mask of only the first 4 bytes (uncheck for exact matching). A progress line shows N / total files with the current file path while scanning; during the scan the options are hidden and the buttons are locked. Rules are created only for files that actually exist on the card (a FCP template is returned); missing files are skipped. Custom files from the Custom files sub-tab are included under the same existence check. The same dialog and options are used by Profile from snapshot, which walks the selected snapshot’s captured files instead of the card; rules for files whose contents were not captured during the scan get no content check (they are reported as unverifiable when the profile is later checked).

    diff --git a/frontend/index.html b/frontend/index.html index 5279b9d..4a233b0 100644 --- a/frontend/index.html +++ b/frontend/index.html @@ -1436,7 +1436,7 @@ // ===== Version ===== // Single source of truth for the PWA version: shown in the header and used // by the server version check in pysimConnect(). -const SIMPLE_VERSION = '3.1.3'; +const SIMPLE_VERSION = '3.1.4'; document.getElementById('app-version').textContent = 'v' + SIMPLE_VERSION; // ===== Tab switching ===== @@ -12964,6 +12964,10 @@ function efFlatten(data) { const rows = []; if (data === null || data === undefined) return rows; if (typeof data !== 'object') { rows.push(['', efPrimitive(data)]); return rows; } + // `empty: true` marks an all-FF (unused) record - a marker, not a field. + // The decoded views show their own "empty" note; the comparison shows + // 'Empty (all FF)' for the empty side of a diff. + if (data.empty === true) return rows; const arrayRows = (label, arr) => { if (!arr.length) return; if (typeof arr[0] !== 'object' || arr[0] === null) { @@ -12997,10 +13001,18 @@ function efDiffData(expData, actData) { for (const [k, v] of efFlatten(expData)) e[k] = v; for (const [k, v] of efFlatten(actData)) a[k] = v; const labels = [...new Set([...Object.keys(e), ...Object.keys(a)])]; + // A side that decodes to an all-FF record has no fields at all: say so in + // its cells instead of leaving them blank. A field merely absent from a + // non-empty record still shows as blank. + const expEmpty = expData && expData.empty === true ? t('Empty (all FF)') : ''; + const actEmpty = actData && actData.empty === true ? t('Empty (all FF)') : ''; const rows = []; for (const l of labels) { const ev = e[l], av = a[l]; - if (ev !== av) rows.push([l, ev === undefined ? '' : ev, av === undefined ? '' : av]); + if (ev === av) continue; + rows.push([l, + ev === undefined ? expEmpty : ev, + av === undefined ? actEmpty : av]); } return rows; } @@ -14036,6 +14048,7 @@ const LANG_RU = { 'not updated': 'не обновлён', 'not present': 'отсутствует', 'empty': 'пусто', + 'Empty (all FF)': 'Пусто (только FF)', 'No data': 'Нет данных', 'Home': 'Домашняя сеть', 'Home equivalent': 'Эквивалентная домашней', diff --git a/frontend/sw.js b/frontend/sw.js index 0ce9f0a..3c6743a 100644 --- a/frontend/sw.js +++ b/frontend/sw.js @@ -1,4 +1,4 @@ -const CACHE = 'simple-v238'; +const CACHE = 'simple-v239'; const URLS = [ 'index.html', 'help.html', diff --git a/frontend/tests/ef_decode.test.js b/frontend/tests/ef_decode.test.js index b421641..c527a43 100644 --- a/frontend/tests/ef_decode.test.js +++ b/frontend/tests/ef_decode.test.js @@ -226,6 +226,30 @@ test('efDiffData reports only differing fields; efContentDiff decodes both sides assert.strictEqual(efContentDiff('EF.NOPE', null, '00', '01'), null); }); +test('efFlatten skips the all-FF empty marker', () => { + assert.deepStrictEqual(efFlatten({ empty: true }), []); +}); + +test('efDiffData names an all-FF side Empty (all FF)', () => { + assert.deepStrictEqual(efDiffData({ empty: true }, { full: 'Miran' }), + [['Full name', 'Empty (all FF)', 'Miran']]); + assert.deepStrictEqual(efDiffData({ full: 'Miran' }, { empty: true }), + [['Full name', 'Miran', 'Empty (all FF)']]); + assert.deepStrictEqual(efDiffData({ empty: true }, { empty: true }), []); + // a field merely missing from a non-empty record stays blank + assert.deepStrictEqual(efDiffData({ full: 'Miran' }, { full: 'Miran', short: 'Mir' }), + [['Short name', '', 'Mir']]); +}); + +test('efContentDiff reports an empty PNN record side', () => { + const expected = 'FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF'; + const actual = '430887CDB43CEC2687FFFFFFFFFFFFFFFFFFFFFF'; + assert.deepStrictEqual(efDecPnn(efBytes(actual)), { full: 'Miran' }); + const dd = efContentDiff('EF.PNN', '6fc5', expected, actual); + assert.ok(dd, 'decoded diff expected'); + assert.deepStrictEqual(dd.rows, [['Full name', 'Empty (all FF)', 'Miran']]); +}); + test('efDataSummary produces one-line summaries', () => { assert.strictEqual(efDataSummary(efDecImsi(efBytes('082982608200002080'))), 'IMSI 228062800000208'); assert.strictEqual(efDataSummary(efDecIccid(efBytes('988812010000400310f0'))), 'ICCID 8988211000000430010'); diff --git a/pyproject.toml b/pyproject.toml index 30b8ec4..09a04d7 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "pysim-simple-server" -version = "3.1.3" +version = "3.1.4" description = "HTTP REST server wrapping pysim for the SIMple PWA" requires-python = ">=3.8" # pysim is a git-only dependency installed explicitly by setup.bat/setup.sh. diff --git a/pysim_simple_server/server.py b/pysim_simple_server/server.py index ea1e38b..357ccac 100644 --- a/pysim_simple_server/server.py +++ b/pysim_simple_server/server.py @@ -28,7 +28,7 @@ from osmocom.tlv import BER_TLV_IE from osmocom.utils import rpad -VERSION = '3.1.3' +VERSION = '3.1.4' MAX_ENVELOPE_SEGMENTS = 5 # max SMS segments for outgoing C-APDU in ENVELOPE