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

soft-deprecate NormalizePath::default in v3 #2529

Merged
merged 2 commits into from
Dec 18, 2021
Merged

soft-deprecate NormalizePath::default in v3 #2529

merged 2 commits into from
Dec 18, 2021

Conversation

robjtede
Copy link
Member

@robjtede robjtede commented Dec 18, 2021

PR Type

Deprecation backport

PR Checklist

  • 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

As a transition step for v4, add warning to v3 NormalizePath middleware noting change in behavior.

Closes #2343

@robjtede robjtede changed the base branch from master to 3.0 December 18, 2021 03:49
@robjtede robjtede added A-web project: actix-web B-semver-patch labels Dec 18, 2021
@robjtede robjtede changed the title sec fixes soft-deprecate NormalizePath in v3 Dec 18, 2021
@robjtede robjtede changed the title soft-deprecate NormalizePath in v3 soft-deprecate NormalizePath::default in v3 Dec 18, 2021
@robjtede robjtede changed the title soft-deprecate NormalizePath::default in v3 soft-deprecate NormalizePath::default in v3 Dec 18, 2021
@robjtede robjtede requested a review from a team December 18, 2021 03:54
@robjtede robjtede marked this pull request as ready for review December 18, 2021 03:57
@robjtede robjtede linked an issue Dec 18, 2021 that may be closed by this pull request
@robjtede robjtede merged commit 0669ed0 into 3.0 Dec 18, 2021
@robjtede robjtede deleted the v3-dep-norm-def branch December 18, 2021 22:57
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-patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TrailingSlash clarification (NormalizePath) on actix_web 3.3.2
1 participant