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

Replace isahc with async ureq #18414

Merged
merged 9 commits into from
Oct 2, 2024
Merged

Replace isahc with async ureq #18414

merged 9 commits into from
Oct 2, 2024

Conversation

ConradIrwin
Copy link
Member

REplace isahc with ureq everywhere gpui is used.

This should allow us to make http requests without libssl; and avoid a long-tail of panics caused by ishac.

Release Notes:

  • (potentially breaking change) updated our http client

This should let us make HTTP requests
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Sep 26, 2024
@maxdeviant maxdeviant changed the title async ureq Replace isahc with async ureq Sep 27, 2024
@mikayla-maki mikayla-maki merged commit 3a5deb5 into main Oct 2, 2024
9 checks passed
@mikayla-maki mikayla-maki deleted the async_ureq branch October 2, 2024 19:30
maxdeviant added a commit that referenced this pull request Oct 2, 2024
This PR reverts the changes to the Clickhouse event rows that were
included in #18414.

The changes don't seem to be correct, as they make the row structs
differ from the underlying table schema.

Release Notes:

- N/A
maxdeviant added a commit that referenced this pull request Oct 2, 2024
This PR sorts the dependencies in various `Cargo.toml` files after
#18414.

Release Notes:

- N/A
schpet added a commit to schpet/zed that referenced this pull request Oct 3, 2024
* upstream/main: (76 commits)
  Remove a debug dev config line (zed-industries#18689)
  Update Rust crate thiserror to v1.0.64 (zed-industries#18677)
  Add command palette action name to outline panel docs (zed-industries#18678)
  Add basic outline panel docs (zed-industries#18674)
  ssh: Add session state indicator to title bar (zed-industries#18645)
  Update Rust crate clap to v4.5.19 (zed-industries#18660)
  Sort dependencies in `Cargo.toml` files (zed-industries#18657)
  collab: Revert changes to Clickhouse event rows (zed-industries#18654)
  Replace isahc with async ureq (zed-industries#18414)
  Update cloudflare/wrangler-action digest to 168bc28 (zed-industries#18651)
  Prepare to sync other kinds of settings (zed-industries#18616)
  editor: Ensure proposed changes editor is syntax-highlighted when opened (zed-industries#18648)
  language: Update buffer doc comments (zed-industries#18646)
  Revert "Fix blurry cursor on Wayland at a scale other than 100%" (zed-industries#18642)
  Tweak warning diagnostic toggle (zed-industries#18637)
  Adjust spacing and sizing of buffer search bar icon buttons (zed-industries#18638)
  docs: Add missing UI font settings to "Configuring Zed" (zed-industries#18267)
  Use `const` over `static` for string literals (zed-industries#18635)
  docs: Add FIPS mode error to Linux troubleshooting (zed-industries#18407)
  v0.157.x dev
  ...
mikayla-maki added a commit that referenced this pull request Oct 9, 2024
noaccOS pushed a commit to noaccOS/zed that referenced this pull request Oct 19, 2024
REplace isahc with ureq everywhere gpui is used.

This should allow us to make http requests without libssl; and avoid a
long-tail of panics caused by ishac.

Release Notes:

- (potentially breaking change) updated our http client

---------

Co-authored-by: Mikayla <[email protected]>
noaccOS pushed a commit to noaccOS/zed that referenced this pull request Oct 19, 2024
This PR reverts the changes to the Clickhouse event rows that were
included in zed-industries#18414.

The changes don't seem to be correct, as they make the row structs
differ from the underlying table schema.

Release Notes:

- N/A
noaccOS pushed a commit to noaccOS/zed that referenced this pull request Oct 19, 2024
This PR sorts the dependencies in various `Cargo.toml` files after
zed-industries#18414.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants