Skip to content

add ServiceIndices field to HandlerType::Wasi0_3#3449

Merged
dicej merged 1 commit intospinframework:mainfrom
dicej:store-indices-in-handlertype-wasi03
Apr 9, 2026
Merged

add ServiceIndices field to HandlerType::Wasi0_3#3449
dicej merged 1 commit intospinframework:mainfrom
dicej:store-indices-in-handlertype-wasi03

Conversation

@dicej
Copy link
Copy Markdown
Contributor

@dicej dicej commented Apr 8, 2026

This allows embedders to efficiently invoke WASIp3 handlers without using wasmtime_wasi_http::handler's instance reuse machinery if desired. Technically, that could be accomplished without this patch by calling ProxyHandler::instance_pre and converting that to a ServiceIndices object, but that's expensive to do on each request, so it's better to do it once during setup.

@dicej dicej force-pushed the store-indices-in-handlertype-wasi03 branch from d563a62 to 5d3ad1f Compare April 8, 2026 23:11
@dicej dicej enabled auto-merge (squash) April 8, 2026 23:11
This allows embedders to efficiently invoke WASIp3 handlers _without_ using
`wasmtime_wasi_http::handler`'s instance reuse machinery if desired.
Technically, that could be accomplished without this patch by calling
`ProxyHandler::instance_pre` and converting that to a `ServiceIndices` object,
but that's expensive to do on each request, so it's better to do it once during
setup.
@dicej dicej force-pushed the store-indices-in-handlertype-wasi03 branch from 5d3ad1f to c130c01 Compare April 8, 2026 23:22
@dicej dicej merged commit 9796cd8 into spinframework:main Apr 9, 2026
17 checks passed
@dicej dicej deleted the store-indices-in-handlertype-wasi03 branch April 9, 2026 00:56
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