Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Добавил методы для параллельной (совмещенной) подписи (CoSign) #45

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

gaarutyunov
Copy link

  1. Добавил два метода:
  • addAttachedSignature - добавляет присоединенную (параллельную) подпись к раннее подписанному документу
  • addDetachedSignature - добавляет отсоединенную (параллельную) подпись к раннее подписанному документу. Основывался на примере
  1. Добавил тесты для этих методов.

Оба варианта работают, проходят проверку через госуслуги

Компилируемые файлы в папках lib и dist в коммит добавлять не стал, чтобы было проще отревьюить. Обновленные примеры, тоже в коммит не добавлял.

Возможно стоит добавить примеры использования, особенно в случае с отсоединенной подписью, чтобы понятно было, что нужно передавать в аргументах подпись и хэш оригинального файла, а не подписи

@gaarutyunov gaarutyunov changed the title Добавил методы для параллельной подписи Добавил методы для параллельной (совмещенной) подписи (CoSign) Mar 23, 2021
@vgoma vgoma reopened this Jan 11, 2023
@khidirbekov
Copy link

@vgoma fyi

@khidirbekov
Copy link

@gaarutyunov

@gaarutyunov
Copy link
Author

@khidirbekov чтобы это влить нужно скорее всего из отдельной ветки пересоздать пулл реквест с CoSign, потому что мы уже давно используем наш форк и там прилично поменялось.

Если нужно, займусь на этой неделе в свободное время.

@khidirbekov
Copy link

@khidirbekov чтобы это влить нужно скорее всего из отдельной ветки пересоздать пулл реквест с CoSign, потому что мы уже давно используем наш форк и там прилично поменялось.

Если нужно, займусь на этой неделе в свободное время.

был бы сильно благодарен

@vgoma
Copy link
Owner

vgoma commented Mar 29, 2024

Спасибо парни, гляну как будет то самое "свободное время" 😅

@khidirbekov
Copy link

@khidirbekov чтобы это влить нужно скорее всего из отдельной ветки пересоздать пулл реквест с CoSign, потому что мы уже давно используем наш форк и там прилично поменялось.

Если нужно, займусь на этой неделе в свободное время.

Привет! Подскажи, пожалуйста, нет апдейтов по этому вопросу?

@khidirbekov
Copy link

fyi @gaarutyunov

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.

4 participants