Skip to content

async signing#422

Merged
kazuho merged 72 commits intomasterfrom
kazuho/async-sign
Dec 4, 2022
Merged

async signing#422
kazuho merged 72 commits intomasterfrom
kazuho/async-sign

Conversation

@kazuho
Copy link
Copy Markdown
Member

@kazuho kazuho commented Sep 26, 2022

PR #408 with some minor tweaks:

  • fix tests
  • make the async case an optional path, realizing that even when the async flag is set, synchronous operation is required when running as client
  • merge cancel_cb and context into one object

sharksforarms and others added 30 commits July 19, 2022 14:49
signing can return `PTLS_ERROR_ASYNC_OPERATION` which will
set the TLS state to `PTLS_STATE_SERVER_GENERATING_CERTIFICATE_VERIFY`

It is up to the contract between the crypto implementation/application to
notify and retry `ptls_handshake`
in the case of asynchronous operation we are waiting on data,
a new buffer cannot be re-used
"The data pointed to by args and of size size will be copied and then passed as an argument to func when the job starts"
https://www.openssl.org/docs/man1.1.1/man3/ASYNC_start_job.html
when set the callback is called in `ptls_free`
proceed as normal if openssl async is not available
Comment thread lib/openssl.c Outdated
Comment thread lib/openssl.c Outdated
@kazuho kazuho mentioned this pull request Oct 6, 2022
@kazuho kazuho merged commit a4c3e53 into master Dec 4, 2022
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.

2 participants