Skip to content
This repository was archived by the owner on Nov 20, 2025. It is now read-only.

fix: pin nock and typescript dependencies to fix window.crypto.subtle.verify signature argument type#2106

Merged
miguelvelezsa merged 1 commit intomainfrom
fix--wrong-type-issue-and-test-timeout-issue
Aug 1, 2025
Merged

fix: pin nock and typescript dependencies to fix window.crypto.subtle.verify signature argument type#2106
miguelvelezsa merged 1 commit intomainfrom
fix--wrong-type-issue-and-test-timeout-issue

Conversation

@miguelvelezsa
Copy link
Copy Markdown
Contributor

Due to a new TS version, we are getting an error in crypto file [1].
Also test are timing out due to a new version in nock.

Set specific versions for both dependencies as short term solution, until we implement the real fix.

[1] src/crypto/browser/crypto.ts:94:7 - error TS2345: Argument of type 'Uint8Array' is not assignable to parameter of type 'BufferSource'.
Type 'Uint8Array' is not assignable to type 'ArrayBufferView'.
Types of property 'buffer' are incompatible.
Type 'ArrayBufferLike' is not assignable to type 'ArrayBuffer'.
Type 'SharedArrayBuffer' is not assignable to type 'ArrayBuffer'.
Types of property '[Symbol.toStringTag]' are incompatible.
Type '"SharedArrayBuffer"' is not assignable to type '"ArrayBuffer"'.

94 signatureArray,
~~~~~~~~~~~~~~

src/crypto/node/crypto.ts:90:3 - error TS2322: Type 'ArrayBuffer | SharedArrayBuffer' is not assignable to type 'ArrayBuffer'.
Type 'SharedArrayBuffer' is not assignable to type 'ArrayBuffer'.
Types of property '[Symbol.toStringTag]' are incompatible.
Type '"SharedArrayBuffer"' is not assignable to type '"ArrayBuffer"'.

90 return buffer.buffer.slice(

@miguelvelezsa miguelvelezsa requested a review from a team August 1, 2025 19:17
@miguelvelezsa miguelvelezsa requested a review from a team as a code owner August 1, 2025 19:17
@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Aug 1, 2025
@miguelvelezsa miguelvelezsa merged commit afc3bc8 into main Aug 1, 2025
21 checks passed
@miguelvelezsa miguelvelezsa deleted the fix--wrong-type-issue-and-test-timeout-issue branch August 1, 2025 19:27
@feywind
Copy link
Copy Markdown
Contributor

feywind commented Aug 1, 2025

Linking for reference: microsoft/TypeScript#14116

We talked about it on chat, but going forward it looks like we can lock to minver safely (~).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants