Skip to content

Make 'ws' feature of actix-http optional in actix-web#3734

Merged
robjtede merged 7 commits intoactix:masterfrom
Porges:master
Aug 29, 2025
Merged

Make 'ws' feature of actix-http optional in actix-web#3734
robjtede merged 7 commits intoactix:masterfrom
Porges:master

Conversation

@Porges
Copy link
Contributor

@Porges Porges commented Aug 19, 2025

PR Type

Other

PR Checklist

  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • A changelog entry has been made for the appropriate packages.
  • Format code with the latest stable rustfmt.
  • (Team) Label with affected crates and semver status.

Overview

Allows users to opt-out of the ws feature of actix-http when depending on actix-web. This allows pruning of packages which are unused or might cause problems in some cases.

→ In my case, the sha1 package may not be depended upon and triggers static analysis failures when it shows up in Cargo.lock.

⚠️ If default-features = false is used and websocket support is required then this is a (build) breaking change.
⚠️ I'm not sure what the correct thing to do with the version bound on actix-web-actors is?

@Porges Porges changed the title Make 'ws' feature of actix-http optional Make 'ws' feature of actix-http optional in actix-web Aug 19, 2025
@robjtede robjtede added B-semver-minor A-web project: actix-web labels Aug 29, 2025
@robjtede robjtede enabled auto-merge August 29, 2025 02:37
@robjtede robjtede disabled auto-merge August 29, 2025 02:38
@robjtede robjtede enabled auto-merge August 29, 2025 02:38
@robjtede robjtede added this pull request to the merge queue Aug 29, 2025
Merged via the queue into actix:master with commit 5041cd1 Aug 29, 2025
13 checks passed
@Porges
Copy link
Contributor Author

Porges commented Oct 23, 2025

@robjtede any chance of a release which includes this?

@JohnTitor
Copy link
Member

@Porges Hey, sorry for the delay! I'm going to cut a new release once a remaining thing is resolved, hopefully in this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-web project: actix-web B-semver-minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants