Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

fix: update capi_crypto_wrappers for browser support #435

Merged
merged 1 commit into from
Nov 24, 2022
Merged

fix: update capi_crypto_wrappers for browser support #435

merged 1 commit into from
Nov 24, 2022

Conversation

tjjfvi
Copy link
Contributor

@tjjfvi tjjfvi commented Nov 24, 2022 •
edited
Loading

Resolves #434

  • For deno & fresh, it inlines the wasm blob in the JS (url fetch wasn't working with fresh) but loads it asynchronously to support browsers' limit on synchronous wasm instantiation
  • For node, it inlines the wasm blob in the JS and loads it synchronously as node doesn't support TLA

@tjjfvi tjjfvi marked this pull request as ready for review November 24, 2022 14:35
@tjjfvi tjjfvi requested a review from harrysolovay as a code owner November 24, 2022 14:35
@tjjfvi tjjfvi requested a review from karl-kallavus November 24, 2022 14:35
@tjjfvi tjjfvi merged commit 1f2f71f into main Nov 24, 2022
@tjjfvi tjjfvi deleted the update-crypto-wrappers branch November 24, 2022 14:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@harrysolovay harrysolovay harrysolovay approved these changes

@karl-kallavus karl-kallavus Awaiting requested review from karl-kallavus

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Blake2_128 fails with WebAssembly.Compile error in browser
2 participants
@tjjfvi @harrysolovay