Skip to content

fix(sdk): limit JWKS cache size on defineSandboxProxy - #309

Merged
QuiiBz merged 2 commits into
mainfrom
sdk/jwks-lru-definesandboxproxy
Aug 20, 2026
Merged

fix(sdk): limit JWKS cache size on defineSandboxProxy#309
QuiiBz merged 2 commits into
mainfrom
sdk/jwks-lru-definesandboxproxy

Conversation

@QuiiBz

@QuiiBz QuiiBz commented Aug 20, 2026

Copy link
Copy Markdown
Member

This PR adds an LRU cache instead of a simple Map for the JWKS cache of defineSandboxProxy to prevent unbounded memory growth. The max size of 100 issuers is arbitrary but a good balance to prevent high memory usage/high number of open sockets while still avoiding to create new issuers all the time

@QuiiBz
QuiiBz requested review from AndyW22 and marc-vercel August 20, 2026 17:56
@vercel

vercel Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sandbox Ready Ready Preview, v0 Aug 20, 2026 6:01pm
sandbox-cli Ready Ready Preview Aug 20, 2026 6:01pm
sandbox-sdk-ai-example Ready Ready Preview Aug 20, 2026 6:01pm
workflow-code-runner Ready Ready Preview Aug 20, 2026 6:01pm

@socket-security

socket-security Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​lru-cache@​10.4.39710010089100

View full report

@QuiiBz
QuiiBz merged commit 2023195 into main Aug 20, 2026
13 checks passed
@QuiiBz
QuiiBz deleted the sdk/jwks-lru-definesandboxproxy branch August 20, 2026 18:43
This was referenced Aug 20, 2026
marc-vercel pushed a commit that referenced this pull request Aug 21, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## sandbox@4.1.0

### Minor Changes

- Add multi-region support:
([#301](#301))

- New `region` and `failoverRegions` options on sandbox create, fork,
and update (SDK), with matching flags on `sandbox create`, `fork`,
`run`, `sh`, and the `sandbox config region` / `sandbox config
failover-regions` commands (CLI).
- New `failoverRegions` getter on `Sandbox` and `regions` getter on
`Snapshot`.
- Regions are now shown in `sandbox ls`, `sessions list`, `snapshots
list`/`get`, `sandbox config list`, and the create/fork summary.

### Patch Changes

- Accept `none` for `--failover-regions` on `sandbox create`, `sandbox
run` and `sandbox fork`.
([#310](#310))

`sandbox config failover-regions <name> none` already clears the list,
but at creation time
`none` was forwarded as a literal region name and rejected by the API.
With a project-level
default failover list there was no way to create a sandbox in one of
those regions from the
CLI, while the SDK and the HTTP API both accept an empty
`failoverRegions` array.

The `none` sentinel is now shared between `create`/`run`/`fork` and
`config`.

- `Sandbox.region` and `Sandbox.failoverRegions` no longer return
`undefined`: the getters now report the platform defaults (`iad1` and
`[]`) when the API omits the fields, so their types are `string` and
`string[]`. The new `DEFAULT_SANDBOX_REGION` export makes the default
region readable. ([#301](#301))

- Updated dependencies
[[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2023195eafe3199c2372f7dad8bfaf0878267c52`](2023195)]:
  - @vercel/sandbox@3.1.0
## @vercel/sandbox@3.1.0

### Minor Changes

- Add multi-region support:
([#301](#301))

- New `region` and `failoverRegions` options on sandbox create, fork,
and update (SDK), with matching flags on `sandbox create`, `fork`,
`run`, `sh`, and the `sandbox config region` / `sandbox config
failover-regions` commands (CLI).
- New `failoverRegions` getter on `Sandbox` and `regions` getter on
`Snapshot`.
- Regions are now shown in `sandbox ls`, `sessions list`, `snapshots
list`/`get`, `sandbox config list`, and the create/fork summary.

- `Sandbox.region` and `Sandbox.failoverRegions` no longer return
`undefined`: the getters now report the platform defaults (`iad1` and
`[]`) when the API omits the fields, so their types are `string` and
`string[]`. The new `DEFAULT_SANDBOX_REGION` export makes the default
region readable. ([#301](#301))

### Patch Changes

- Update `defineSandboxProxy` to limit the JWKS issuer cache size to
prevent unbounded memory usage.
([#309](#309))
## @vercel/sandbox-mock@3.1.0

### Minor Changes

- Add multi-region support:
([#301](#301))

- New `region` and `failoverRegions` options on sandbox create, fork,
and update (SDK), with matching flags on `sandbox create`, `fork`,
`run`, `sh`, and the `sandbox config region` / `sandbox config
failover-regions` commands (CLI).
- New `failoverRegions` getter on `Sandbox` and `regions` getter on
`Snapshot`.
- Regions are now shown in `sandbox ls`, `sessions list`, `snapshots
list`/`get`, `sandbox config list`, and the create/fork summary.

- `Sandbox.region` and `Sandbox.failoverRegions` no longer return
`undefined`: the getters now report the platform defaults (`iad1` and
`[]`) when the API omits the fields, so their types are `string` and
`string[]`. The new `DEFAULT_SANDBOX_REGION` export makes the default
region readable. ([#301](#301))

### Patch Changes

- Updated dependencies
[[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2023195eafe3199c2372f7dad8bfaf0878267c52`](2023195)]:
  - @vercel/sandbox@3.1.0

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
marc-vercel pushed a commit that referenced this pull request Aug 21, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to drives-beta, this
PR will be updated.

⚠️⚠️⚠️⚠️⚠️⚠️

`drives-beta` is currently in **pre mode** so this branch has
prereleases rather than normal releases. If you want to exit
prereleases, run `changeset pre exit` on `drives-beta`.

⚠️⚠️⚠️⚠️⚠️⚠️

# Releases
## sandbox@4.1.0-beta.1

### Minor Changes

- Add multi-region support for drives:
([#307](#307))

- Set the `region` field when creating a new drive, or use the
`--region` flag in the CLI.
- You can retrieve the region of a drive using the `region` getter on
the `Drive` class.
- Drives can be only be mounted to sandboxes running on the same region.

- Add multi-region support:
([#301](#301))

- New `region` and `failoverRegions` options on sandbox create, fork,
and update (SDK), with matching flags on `sandbox create`, `fork`,
`run`, `sh`, and the `sandbox config region` / `sandbox config
failover-regions` commands (CLI).
- New `failoverRegions` getter on `Sandbox` and `regions` getter on
`Snapshot`.
- Regions are now shown in `sandbox ls`, `sessions list`, `snapshots
list`/`get`, `sandbox config list`, and the create/fork summary.

### Patch Changes

- `Sandbox.region` and `Sandbox.failoverRegions` no longer return
`undefined`: the getters now report the platform defaults (`iad1` and
`[]`) when the API omits the fields, so their types are `string` and
`string[]`. The new `DEFAULT_SANDBOX_REGION` export makes the default
region readable. ([#301](#301))

- Updated dependencies
[[`c2b5d7b45340b485264abb82f0cd640ed4e93293`](c2b5d7b),
[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2023195eafe3199c2372f7dad8bfaf0878267c52`](2023195)]:
  - @vercel/sandbox@3.1.0-beta.1
## @vercel/sandbox@3.1.0-beta.1

### Minor Changes

- Add multi-region support for drives:
([#307](#307))

- Set the `region` field when creating a new drive, or use the
`--region` flag in the CLI.
- You can retrieve the region of a drive using the `region` getter on
the `Drive` class.
- Drives can be only be mounted to sandboxes running on the same region.

- Add multi-region support:
([#301](#301))

- New `region` and `failoverRegions` options on sandbox create, fork,
and update (SDK), with matching flags on `sandbox create`, `fork`,
`run`, `sh`, and the `sandbox config region` / `sandbox config
failover-regions` commands (CLI).
- New `failoverRegions` getter on `Sandbox` and `regions` getter on
`Snapshot`.
- Regions are now shown in `sandbox ls`, `sessions list`, `snapshots
list`/`get`, `sandbox config list`, and the create/fork summary.

- `Sandbox.region` and `Sandbox.failoverRegions` no longer return
`undefined`: the getters now report the platform defaults (`iad1` and
`[]`) when the API omits the fields, so their types are `string` and
`string[]`. The new `DEFAULT_SANDBOX_REGION` export makes the default
region readable. ([#301](#301))

### Patch Changes

- Update `defineSandboxProxy` to limit the JWKS issuer cache size to
prevent unbounded memory usage.
([#309](#309))
## @vercel/sandbox-mock@3.1.0-beta.1

### Minor Changes

- Add multi-region support:
([#301](#301))

- New `region` and `failoverRegions` options on sandbox create, fork,
and update (SDK), with matching flags on `sandbox create`, `fork`,
`run`, `sh`, and the `sandbox config region` / `sandbox config
failover-regions` commands (CLI).
- New `failoverRegions` getter on `Sandbox` and `regions` getter on
`Snapshot`.
- Regions are now shown in `sandbox ls`, `sessions list`, `snapshots
list`/`get`, `sandbox config list`, and the create/fork summary.

- `Sandbox.region` and `Sandbox.failoverRegions` no longer return
`undefined`: the getters now report the platform defaults (`iad1` and
`[]`) when the API omits the fields, so their types are `string` and
`string[]`. The new `DEFAULT_SANDBOX_REGION` export makes the default
region readable. ([#301](#301))

### Patch Changes

- Add mock for the `Drive` class.
([#307](#307))

- Updated dependencies
[[`c2b5d7b45340b485264abb82f0cd640ed4e93293`](c2b5d7b),
[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2997ecc32a77c145289fbbd89ae2dc0214767ae3`](2997ecc),
[`2023195eafe3199c2372f7dad8bfaf0878267c52`](2023195)]:
  - @vercel/sandbox@3.1.0-beta.1

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants