Skip to content

script: Implement "getPublicKey" operations of RSA algorithms#43093

Merged
Taym95 merged 1 commit intoservo:mainfrom
kkoyung:get-public-key-rsa
Mar 8, 2026
Merged

script: Implement "getPublicKey" operations of RSA algorithms#43093
Taym95 merged 1 commit intoservo:mainfrom
kkoyung:get-public-key-rsa

Conversation

@kkoyung
Copy link
Copy Markdown
Member

@kkoyung kkoyung commented Mar 8, 2026

Implement "getPublicKey" operations of RSASSA-PKCS1-v1_5, RSA-PSS and RSA-OAEP. The steps are implemented at rsa_common.rs shared by the three RSA algorithms.

To make the operations functioning, they are also registered at GetPublicKeyAlgorithm.

Specification: https://wicg.github.io/webcrypto-modern-algos/#SubtleCrypto-method-getPublicKey
Testing: Pass WPT tests that were expected to fail.
Fixes: Part of #43072

Implement "getPublicKey" operations of RSASSA-PKCS1-v1_5, RSA-PSS and
RSA-OAEP. The steps are implemented at `rsa_common.rs` shared by the
three RSA algorithms.

To make the operations functioning, they are also registered at
`GetPublicKeyAlgorithm`.

Specification: https://wicg.github.io/webcrypto-modern-algos/#SubtleCrypto-method-getPublicKey
Testing: Pass WPT tests that were expected to fail.
Fixes: Part of 43072

Signed-off-by: Kingsley Yung <[email protected]>
@kkoyung kkoyung requested a review from gterzian as a code owner March 8, 2026 13:07
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 8, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 8, 2026
@Taym95 Taym95 added this pull request to the merge queue Mar 8, 2026
@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 Mar 8, 2026
Merged via the queue into servo:main with commit d472788 Mar 8, 2026
33 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 Mar 8, 2026
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