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

Accurate CryptographicUsageMask for KMIP creation of RSA keys #189

Conversation

JosePisco
Copy link
Contributor

Same as #187 but for RSA keys. Code enhancement from this PR has been updated here since changes are relatively similar.

@JosePisco JosePisco requested review from ThibsG and Manuthor February 20, 2024 15:04
@JosePisco JosePisco linked an issue Feb 20, 2024 that may be closed by this pull request
…on-on-elliptic-curves' into 186-accurate-cryptographicusagemask-for-kmip-creation-of-rsa-keys
* feat: harmonising crypto consts in bits

* feat: style + load provider in fips tests

* feat: load fips provider on rsa tests

* fix: better const naming for passwd derivation
@Manuthor Manuthor merged commit 3b533a6 into 151-accurate-cryptographicalgorithm-for-kmip-key-creation-on-elliptic-curves Mar 1, 2024
@Manuthor Manuthor deleted the 186-accurate-cryptographicusagemask-for-kmip-creation-of-rsa-keys branch March 1, 2024 15:49
Manuthor pushed a commit that referenced this pull request Mar 1, 2024
…ic curves and RSA (#187 and #189)

* Move crypto subcrate into kmip and dispatch other elements

* Remove unused feature curve25519

* Set openssl as wanted feature for kmip deps in pyo3

* Update patched iana-time-zone to a non yanked version

* feat: better crypto const organization

* feat: better zeroization

* feat: finalize zeroization

* revert type in kmip operation back to vec<u8>

* fix build

* revert commit reverting zeroizing + modification attempt at serializer

* Temporarily ignore rustsec from pqc-kyber

* feat: code enhancement deserialize

* feat: dynamic cryptographic algorithm and usage for creation. Tests fail

* feat: dynami mask and cryptographic algorithm for ecc key creation

* fix: loose comparison

* feat: tests + refacto dynamic mask and algo for ecc key creation

* fix: fix compile from bad merge

* fix lint

* feat: distinguish private and public key masks

* fix: remove redundant refrence

* feat: code enhancement

* fix: unused variable in non-fips mode

* feat: style + load provider in fips tests

* fix: remove P-192 from fips mask construction

* feat: accurate CryptographicUsageMask for KMIP creation of RSA keys (#189)

* feat: dynamic cryptographic usage mask set and check

* feat: code enhancement

* feat: style + load provider in fips tests

* feat: load fips provider on rsa tests

* fix: inconsistent crypto consts (#190)

* feat: harmonising crypto consts in bits

* feat: style + load provider in fips tests

* feat: load fips provider on rsa tests

* fix: better const naming for passwd derivation

---------

Co-authored-by: ThibsG <thibsg@pm.me>
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.

Accurate CryptographicUsageMask for KMIP creation of RSA keys
3 participants