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

Support GOST 2012 256 with botan #616

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from
Draft

Conversation

gecheim
Copy link

@gecheim gecheim commented Mar 11, 2021

In these commits the following changes:

  • Fix build with CMake
  • Added support CMake build with static botan library
  • Added support work with GOST R 34.10 2012 256 keys with botan backend.

Issue #404

@rijswijk
Copy link
Contributor

Thank you for the PR; I gave it a quick review, I saw some changes in the OpenSSL code base as well, the PR suggests it is only for Botan, do I see correctly that these are only there to make the code path for getGOSTType(..) complete? If so, is this the right approach since it seems GOST 2012 is not actually supported with OpenSSL?

I will also ask @halderen for a more detailed review.

@rijswijk rijswijk requested a review from halderen March 11, 2021 15:44
@gecheim
Copy link
Author

gecheim commented Mar 11, 2021

Yes, changes in OpenSSL code base support only getGOSTType(..) otherwise the project would not build with the OpenSSL backend.

OpenSSL support GOST 2012 with engine.

These changes were made for a task where you only need to work with the botan backend. So the other option was not supported in this pull request.

@atribolt
Copy link

We have the same problem. We need to work with keys like Gost2012-256 using botan. Could you apply this patch to support working with botan? But for Gost 2012 with openssl, create a separate issue.

@jschlyter jschlyter marked this pull request as draft November 29, 2024 16:24
@jschlyter
Copy link
Contributor

Please rebase on develop and mark as ready when ready.

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