search

emscripten-forge / requests-wasm-polyfill

Drop-in replacement for the requests library for wasm python
9 stars 2 forks source link

Patch `urllib.urlopen` #4

Closed jtpio closed 7 months ago

jtpio commented 1 year ago

It would be nice to also patch urllib.urlopen by default as it is also commonly used.

For example pyodide-http supports it: https://github.com/koenvo/pyodide-http#supported-packages

image

jtpio commented 1 year ago

Or maybe this could be a separate repo / package under emscripten-forge/requests-urllib-polyfill?

DerThorsten commented 1 year ago

Best would be to make https://github.com/koenvo/pyodide-http usable with pyjs remove this repo

jtpio commented 1 year ago

Sounds good.

jonas-w commented 11 months ago

@DerThorsten is there something that blocks porting pyodide-http to emscripten? Currently, just used pyodide and wanted to try out the jupyterlite-xeus kernel, but the requests package on emscripten breaks many things for me, compared to pyodide where most stuff worked after some tweaks.

jtpio commented 7 months ago

Closing as this is not needed anymore: https://github.com/emscripten-forge/recipes/issues/782