Cloudflare workers bug since 0.11

[VictorBulba]

Hey!

Looks like something is broken in Cloudflare workers runtime.

Minimal example:

#[wasm_bindgen]
pub async fn handle() -> String {
    let resp = match reqwest::get("https://wtfismyip.com/text").await {
        Ok(resp) => resp,
        Err(err) => return err.to_string(),
    };
    let text = resp.text().await.unwrap();
    text
}

With version 0.10 it works, but with 0.11 it falls into Err arm of the match statement.

Error message:

builder error: JsValue(Error: The 'credentials' field on 'RequestInitializerDict' is not implemented.
Error: The 'credentials' field on 'RequestInitializerDict' is not implemented.
    at worker.js:343:19
    at worker.js:222:22
    at <anonymous>:wasm-function[351]:0x36dc7
    at <anonymous>:wasm-function[49]:0xb3c4
    at <anonymous>:wasm-function[208]:0x31f93
    at <anonymous>:wasm-function[188]:0x30b0d
    at _dyn_core__ops__function__FnMut__A____Output___R_as_wasm_bindgen__closure__WasmClosure___describe__invoke__h0dedee9324219338 (<anonymous>:wasm-function[435]:0x37e0e)
    at __wbg_adapter_26 (worker.js:208:10)
    at real (worker.js:193:20))

[seanmonstar]

Hm, that's probably because of #1227. Perhaps a fix is that we could change reqwest to have that internal type be an Option<RequestCredentials>, and so by default if you never set one, it should no longer try to set that field.

[bensadiku]

I'll give it a shot :)

[bensadiku]

@VictorBulba I've opened #1249, can you confirm it's fixed?

[VictorBulba]

@bensadiku it works, thanks!