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

Jump to bottom

add Route::wrap #2725

Merged
merged 5 commits into from Apr 23, 2022
Merged

add Route::wrap #2725

merged 5 commits into from Apr 23, 2022

Conversation

robjtede
Copy link
Member

@robjtede robjtede commented Apr 8, 2022 •
edited

PR Type

Feature

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

Wrapping single handlers was not previously possible without wrapping the whole Route which would cause the middleware to run before guards.

@robjtede robjtede added C-feature Category: new functionality A-web project: actix-web labels Apr 8, 2022
@robjtede robjtede added this to the actix-web v4.1 milestone Apr 8, 2022
@robjtede robjtede added B-semver-minor and removed C-feature Category: new functionality labels Apr 8, 2022
@robjtede robjtede marked this pull request as ready for review April 8, 2022 12:02
@robjtede robjtede enabled auto-merge (squash) April 23, 2022 19:33
@robjtede robjtede disabled auto-merge April 23, 2022 20:01
@robjtede robjtede merged commit 45592b3 into master Apr 23, 2022
@robjtede robjtede deleted the route-wrap branch April 23, 2022 20:01
@robjtede robjtede mentioned this pull request Apr 27, 2022
Closed
@lermana lermana mentioned this pull request Mar 23, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
A-web project: actix-web B-semver-minor
Projects
None yet
Milestone
actix-web v4.1
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@robjtede