Skip to content

Clarify the required behavior of concurrent calls#142

Merged
athoelke merged 4 commits into
ARM-software:mainfrom
athoelke:crypto-concurrent
Jan 12, 2024
Merged

Clarify the required behavior of concurrent calls#142
athoelke merged 4 commits into
ARM-software:mainfrom
athoelke:crypto-concurrent

Conversation

@athoelke
Copy link
Copy Markdown
Contributor

Relax/clarify the statement regarding the required behavior of implementations that support concurrent execution of calls to the API.

Fixes #134

@athoelke athoelke added clarification Something is confusing or missing in the documentation Crypto API Issue or PR related to the Cryptography API labels Jan 10, 2024
@athoelke athoelke added this to the Crypto API 1.2 milestone Jan 10, 2024
Copy link
Copy Markdown

@Ryan-Everett-arm Ryan-Everett-arm left a comment

Choose a reason for hiding this comment

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

It was a good idea to add an example here, the intuition of concurrency constraints can be difficult to understand. I'm not sure if the proposed constraint will be interpreted as intended, though; see my comment below.

Comment thread doc/crypto/overview/conventions.rst Outdated
* Provide general caveat about consistency and resource-related errors
* Add some structure to the concurrency rules
Copy link
Copy Markdown

@Ryan-Everett-arm Ryan-Everett-arm left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread doc/crypto/overview/conventions.rst Outdated
@athoelke athoelke merged commit 4e21ae7 into ARM-software:main Jan 12, 2024
@athoelke athoelke deleted the crypto-concurrent branch January 12, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clarification Something is confusing or missing in the documentation Crypto API Issue or PR related to the Cryptography API

Projects

Development

Successfully merging this pull request may close these issues.

Crypto: soften the requirement for concurrency

3 participants