Skip to content

webserver: migrate to using State instead of extensions - #3558

Merged
syphar merged 1 commit into
rust-lang:mainfrom
syphar:axum-state
Sep 30, 2026
Merged

syphar merged 1 commit into
rust-lang:mainfrom
syphar:axum-state

Conversation

@syphar

@syphar syphar commented Sep 24, 2026

Copy link
Copy Markdown
Member

this is the official axum way of sharing state with handlers.

We only didn't do it because it didn't exist when I did the migration, and only came a couple of months later :)

@syphar syphar self-assigned this Sep 24, 2026
@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Sep 24, 2026
@syphar
syphar marked this pull request as ready for review September 30, 2026 11:49
@syphar
syphar requested a review from a team as a code owner September 30, 2026 11:49
@syphar

syphar commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

r? @GuillaumeGomez

@rustbot rustbot assigned GuillaumeGomez and unassigned syphar Sep 30, 2026
Comment thread crates/bin/docs_rs_web/src/state.rs
@syphar
syphar enabled auto-merge (rebase) September 30, 2026 12:39
@syphar
syphar disabled auto-merge September 30, 2026 12:53
@syphar
syphar merged commit 05e43f0 into rust-lang:main Sep 30, 2026
15 checks passed
@syphar
syphar deleted the axum-state branch September 30, 2026 12:53
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Sep 30, 2026
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Sep 30, 2026

This branch was successfully deployed

1 active deployment
github-pages — 547b9e48 Deployed Sep 30, 2026 by syphar via deploy #582
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

Successfully merging this pull request may close these issues.

2 participants