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

Abstract perform_https_request so that it can be used in Wasm context #35

Open
carlopi opened this issue Oct 30, 2024 · 1 comment
Open

Comments

@carlopi
Copy link

carlopi commented Oct 30, 2024

This is something I touched on recently while looking at what would it take to make possible for https://github.com/duckdb/uc_catalog to work in a Web context.

Problem is basically how to wrap a minimal API that allows to make requests either natively OR from WebAssembly (there Web and Node might be two slightly different challenges).

I think I know what's the way forward, and implementing a curl wrapper (say as a header) would allow to solve the problems in:

  • duckdb/uc_catalog
  • evidence-dev/duckdb_gsheets
    and possibly elsewhere.

I think it's likely on me, I can ping when I have a solution.

Given it's currently not functional, I opened a PR to disable Wasm, @archiewood, WDYT? (duckdb/community-extensions#169)

@archiewood
Copy link
Member

Agreed, I would love to bring this to WASM.

I'm not particularly happy with the implementation of perform_http_request() using OpenSSL as it is. It feels pretty dirty.

I was experimenting with libcurl but couldn't get it to work

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

No branches or pull requests

2 participants