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

WASM implemention is missing basic_auth(…) #1095

Closed
ctron opened this issue Nov 30, 2020 · 3 comments
Closed

WASM implemention is missing basic_auth(…) #1095

ctron opened this issue Nov 30, 2020 · 3 comments

Comments

@ctron
Copy link
Contributor

ctron commented Nov 30, 2020

I looks like the WASM version of the request builder is missing the basic_auth function:

Which results e.g. in:

   Compiling openid v0.7.0
error[E0599]: no method named `basic_auth` found for struct `reqwest::RequestBuilder` in the current scope
   --> /home/jreimann/.cargo/registry/src/github.com-1ecc6299db9ec823/openid-0.7.0/src/client.rs:490:14
    |
490 |             .basic_auth(&self.client_id, Some(&self.client_secret))
    |              ^^^^^^^^^^ method not found in `reqwest::RequestBuilder`
ctron added a commit to ctron/reqwest that referenced this issue Nov 30, 2020
Signed-off-by: Jens Reimann <jreimann@redhat.com>
ctron added a commit to ctron/reqwest that referenced this issue Dec 1, 2021
Signed-off-by: Jens Reimann <jreimann@redhat.com>
ctron added a commit to ctron/reqwest that referenced this issue Dec 1, 2021
Signed-off-by: Jens Reimann <jreimann@redhat.com>
@crochee
Copy link

crochee commented Mar 18, 2022

I also encountered the problem

@crochee
Copy link

crochee commented Mar 18, 2022

Cargo.toml is written with [target."cfg(not(target_arch = \"wasm32\"))".dependencies.base64] version = "0.13"

seanmonstar pushed a commit that referenced this issue May 30, 2022
Signed-off-by: Jens Reimann <jreimann@redhat.com>
@ctron
Copy link
Contributor Author

ctron commented Jul 1, 2022

I guess this can be closed as #1096 is merged and released.

@ctron ctron closed this as completed Jul 1, 2022
ctron added a commit to drogue-iot/drogue-client that referenced this issue Jul 1, 2022
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