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

Add _ecVrfy and _ecSign #31

Merged

Conversation

ffontaine
Copy link
Contributor

Add _ecVrfy and _ecSign to allow the user to customize the verification
and signature functions (could be useful if the user wants to use
BearSSLClient without ECC508)

The functions can be customized through the constructor and are set to
their current values as soon as the user calls setEccSlot

Signed-off-by: Fabrice Fontaine fabrice.fontaine@orange.com

Add _ecVrfy and _ecSign to allow the user to customize the verification
and signature functions (could be useful if the user wants to use
BearSSLClient without ECC508)

The functions can be customized through the constructor and are set to
their current values as soon as the user calls setEccSlot

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
@github-actions
Copy link

Memory usage change @436c232c451dc4ad7a1ce0ebd606cfbb4b21699b

FQBN Flash Usage RAM For Global Variables
arduino:megaavr:uno2018 0 0
arduino:samd:mkr1000 0 0
arduino:samd:mkrgsm1400 0 0

Copy link
Contributor

@aentinger aentinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ffontaine 👋 Thank you very much for your contribution. I've been hesitant so far to merge it since it would require changes in our ArduinoIoTCloud library. However, since ArduinoIoTCloud does no longer depend on ArduinoBearSSL this change is good to merge 👍

@aentinger aentinger merged commit 996778e into arduino-libraries:master Sep 17, 2020
@ffontaine ffontaine deleted the _ecVrfy_and__ecSign branch September 17, 2020 08:14
@ffontaine
Copy link
Contributor Author

Thanks for merging this PR, I sent another small one to also allow the user to set its own certificate

@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants