Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: prewarm wasm #2727

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

perf: prewarm wasm #2727

wants to merge 2 commits into from

Conversation

e-moran
Copy link
Contributor

@e-moran e-moran commented Jan 6, 2025

This change will pre-warm the wasm cache once the user creates their arcjet object.

We await the pre-warm promise inside of protect() so that we don't kick off another wasm load by calling fingerprint first.

I added prewarmWasmCache() to analyze so that we didn't have to expose initializeWasm(), which would bring the wasm component in to its public api.

@e-moran e-moran requested a review from blaine-arcjet January 6, 2025 16:52
@e-moran e-moran requested a review from a team as a code owner January 6, 2025 16:52
Copy link

trunk-io bot commented Jan 6, 2025

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

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.

1 participant