Skip to content

NOISSUE - Add wasm http url#169

Open
JeffMboya wants to merge 11 commits intoabsmach:mainfrom
JeffMboya:feat/wasm-http-url
Open

NOISSUE - Add wasm http url#169
JeffMboya wants to merge 11 commits intoabsmach:mainfrom
JeffMboya:feat/wasm-http-url

Conversation

@JeffMboya
Copy link
Copy Markdown
Contributor

@JeffMboya JeffMboya commented Mar 23, 2026

What type of PR is this?

This is a feature - adds wasm HTTP URL delivery

What does this do?

Proplets can now fetch wasm binaries via plain HTTP URL. Adds wasm_http_url field to tasks, with a 100MB streaming size limit and URL scheme validation

Which issue(s) does this PR fix/relate to?

  • No related issue

Have you included tests for your changes?

Yes - unit tests for fetch logic and SQLite round-trips

Did you document any new/modified features?

Documented at absmach/propeller-website#30

Notes

Adds wasm_http_url field to StartRequest. When set, the proplet fetches
the wasm binary directly via HTTP instead of MQTT chunking or OCI registry.
Enforces a 100MB streaming size limit and validates the URL scheme.
@JeffMboya JeffMboya force-pushed the feat/wasm-http-url branch from e66dc6a to eb9509c Compare March 23, 2026 11:01
Comment thread manager/service.go Outdated
"name": t.Name,
"state": t.State,
"image_url": t.ImageURL,
"wasm_http_url": t.WasmHTTPURL,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on the image_url, can we identify if the wasm module is in a registry or not?

For example:

  • docker.io/rodneydav/addition.wasm -> registry
  • https://propeller.absmach.eu/examples/addition.wasm -> Blob

…separate field

Remove the wasm_http_url field from Task and StartRequest. Proplets now detect
plain HTTP delivery by checking if image_url starts with http:// or https://,
falling through to registry fetch otherwise. Drops the DB migration and storage
columns added for wasm_http_url.
JeffMboya added a commit to JeffMboya/propeller that referenced this pull request Mar 26, 2026
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