Skip to content

Revert Key Objects Data validation #54

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

Merged
merged 2 commits into from
May 13, 2024
Merged

Conversation

tgonzalezorlandoarm
Copy link
Member

The current implementation of OSSL_FUNC_KEYMGMT_VALIDATE is incomplete and requires some work.
The TLS handshake on openssl won't require this functionality being implemented.
As a result, OSSL_FUNC_KEYMGMT_VALIDATE won't be finished in the near future, until we implement we have the TLS handshake working.

  • Remove the implementation of the validation function as the current implementation is too incomplete and not removing it may bring more maintainance effort in the future.
  • Remove the unit test for the validation function as the current implementation is too incomplete and not removing it may bring more maintainance effort in the future. The test can be re-enabled and altered when the correct implementation is performed.

This reverts commits 155b860 and 9e16fec.

The current implementation of OSSL_FUNC_KEYMGMT_VALIDATE is
incomplete and requires some work.
The TLS handshake on openssl won't require this functionality being
implemented.
As a result, OSSL_FUNC_KEYMGMT_VALIDATE won't be finished in the near
future, until we implement we have the TLS handshake working.

 * Remove the implementation of the validation function as the
   current implementation is too incomplete and not removing it
   may bring more maintainance effort in the future.

This reverts commit 155b860.

Signed-off-by: Tomás González <tomasagustin.gonzalezorlando@arm.com>
The current implementation of OSSL_FUNC_KEYMGMT_VALIDATE is
incomplete and requires some work.
The TLS handshake on openssl won't require this functionality being
implemented.
As a result, OSSL_FUNC_KEYMGMT_VALIDATE won't be finished in the
near future, until we implement we have the TLS handshake working.

 * Remove the unit test for the validation function as the
   current implementation is too incomplete and not removing it
   may bring more maintainance effort in the future. The test
   can be re-enabled and altered when the correct implementation
   is performed.

This reverts commit 9e16fec.

Signed-off-by: Tomás González <tomasagustin.gonzalezorlando@arm.com>
@gowthamsk-arm gowthamsk-arm merged commit a4ae97a into main May 13, 2024
@tgonzalezorlandoarm tgonzalezorlandoarm deleted the tg/disable-validate branch May 20, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
small Effort label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants