Skip to content

script: Add error messages to operations of ChaCha20-Poly1305#41030

Merged
TimvdLippe merged 1 commit intoservo:mainfrom
kkoyung:chacha20poly1305-error-message
Dec 3, 2025
Merged

script: Add error messages to operations of ChaCha20-Poly1305#41030
TimvdLippe merged 1 commit intoservo:mainfrom
kkoyung:chacha20poly1305-error-message

Conversation

@kkoyung
Copy link
Copy Markdown
Member

@kkoyung kkoyung commented Dec 3, 2025

Error messages in the import/export key operations of ChaCha20-Poly1305 are missing. This patch adds those error messages.

Testing: No behavioral change. Existing tests suffice.

I forgot to provide the error messages in 40948, which implemented the
import/export key operations of ChaCha20-Poly1305. This patch adds back
the error messages.

Testing: No behavioral change. Existing tests suffice.

Signed-off-by: Kingsley Yung <[email protected]>
@kkoyung kkoyung requested a review from gterzian as a code owner December 3, 2025 12:08
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Dec 3, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Dec 3, 2025
@TimvdLippe TimvdLippe added this pull request to the merge queue Dec 3, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Dec 3, 2025
Merged via the queue into servo:main with commit f19bffd Dec 3, 2025
32 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Dec 3, 2025
@kkoyung kkoyung deleted the chacha20poly1305-error-message branch December 8, 2025 11:20
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.

4 participants