SignService 1.3.0: соподписание по содержимому документа и полная проверка ГОСТ-подписей КриптоПро

[zaynullinmi]

Зачем

1. Ошибка в 1.2.0. КриптоПро и CryptoAPI (включая саму SignService на Windows) записывают в signatureAlgorithm OID ГОСТ-ключа с параметрами, а BouncyCastle такие подписи отклоняет. Поэтому реальные ГОСТ-подписи получали «не определено» и при объединении помечались как непроверенные.
2. Соподписание. Ветка форка sozertsatel/SignService (cursor/cosign-discovery-gost-cb6f) добавляет поиск подписей по содержимому документа. Идея нужная, но в исходном виде в ветке были проблемы, найденные при ревью:
   • обход проверки: подпись без messageDigest принималась для любого документа;
   • зависание интерфейса при добавлении файлов;
   • перезапись подписей оригинала в режиме «подписывать копию со штампом»;
   • жёсткие пути в тестах.
3. Ревью форка debug23win (PR Исправить обработку подписей и МЧД; добавить разделение и удаление подписанта #2). Присланные сценарии проверены на текущем коде и закреплены тестами.

Что сделано

Проверка ГОСТ

• На проверку идёт копия подписи без параметров в signatureAlgorithm, OID ключа заменён на OID алгоритма подписи. Это поле не входит в подписанные данные. Все проверки BouncyCastle (messageDigest, content-type, значение подписи) сохраняются, исходные байты подписантов при объединении не меняются.
• При подписанных атрибутах без messageDigest документ считается несоответствующим.
• Покрыты ГОСТ 2001, 2012-256 и 2012-512 в трёх вариантах: OID подписи с параметрами, OID ключа с параметрами (как CryptoAPI), OID ключа без параметров. Повреждённые подписи, другой документ и подписи без messageDigest отклоняются.

Соподписание по содержимому (по ветке форка, с доработками)

• Подписанты сопоставляются той же криптопроверкой, что и при объединении, поэтому прогноз исключений совпадает с результатом.
• «документ.sig» предлагается всегда, поэтому голый PKCS#7 и подписи с неизвестным хешем не теряются.
• Поиск идёт в фоне, документ читается и хешируется один раз. 10 документов по 10 МБ плюс подписи: было 7,6 с в потоке интерфейса, стало 17 мс. Подписание ждёт окончания поиска.
• Перетащенные подписи становятся приложенными: результат пишется рядом с документом, чужие файлы не перезаписываются.
• Резервная копия создаётся только при замене файла без объединения или при исключении подписантов.
• Выбор режима блокируется на время подписания. После подписания варианты обновляются для следующего соподписания.
• «Подписывать копию со штампом» не трогает подписи оригинала. В форке групповая подпись оригинала здесь перезаписывалась.
• Из форка также перенесены подтверждения, строка плана, итог «9 (8 + ваша)» и имя результата «Объединить .sig».
• Workflow test-latest и путь /opt/cursor в тестах не перенесены.

Тесты по ревью форка debug23win

• Изменённое значение подписи при верном messageDigest исключается: в «Объединить .sig», в «Собрать контейнер» и при соподписании.
• Сертификат, который был только в контейнере исключённого подписанта, сохраняется.
• МЧД без номера или даты выдачи, а также с просроченным или недоверенным самоподписанным сертификатом руководителя не проходит как «Ok».

Проверка

• dotnet build: 0 предупреждений.
• dotnet run --project tests/SignService.Tests -c Release — ALL TESTS PASSED, включая хранилище сертификатов и смоук интерфейса (строка соподписания).
• Мутационные проверки: регрессионный тест падает, если вернуть старое поведение. Проверено для режима копии со штампом.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BtHSRXSuM8qTBTUBe29624

---

Generated by Claude Code