make TrailingSlash enum accessible

[Lesiuk]

PR Type

Bug Fix

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

Overview

Wanted to try middleware::NormalizePath::new(TrailingSlash::Trim) introduced in 3.0.0 but I noticed that TrailingSlash enum is inside private package and It's not currently possible.

[codecov-commenter]

Codecov Report

Merging #1673 into master will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1673      +/-   ##
==========================================
+ Coverage   53.29%   53.43%   +0.14%     
==========================================
  Files         123      121       -2     
  Lines       11856    11853       -3     
==========================================
+ Hits         6319     6334      +15     
+ Misses       5537     5519      -18     

Impacted Files	Coverage Δ
tests/test_server.rs	100.00% <100.00%> (ø)
actix-http/src/header/common/accept.rs	0.00% <0.00%> (ø)
actix-http/src/header/common/accept_language.rs
actix-http/src/header/common/allow.rs
actix-http/src/header/common/content_language.rs
actix-http/src/header/common/if_none_match.rs
...ctix-http/src/header/common/if_unmodified_since.rs	0.00% <0.00%> (ø)
actix-http/src/header/common/if_match.rs	0.00% <0.00%> (ø)
actix-http/src/h1/decoder.rs	63.23% <0.00%> (+1.37%)	⬆️
actix-http/src/header/common/mod.rs	10.00% <0.00%> (+2.30%)	⬆️
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64a2c13...33ca882. Read the comment docs.

[robjtede]

Needs a cargo fmt.

Can we add a test to tests/test_server.rs something like:

#[actix_rt::test]
async fn test_normalize() {
    let srv = test::start_with(test::config().h1(), || {
        App::new()
            .wrap(Compress::new(NormalizePath::new(TrailingSlash::Trim)))
            .service(web::resource("/one").route(web::to(|| HttpResponse::Ok().finish())))
    });

    let mut response = srv
        .get("/one/")
        .send()
        .await
        .unwrap();
    assert!(response.status().is_success());
}

[robjtede]

Just a changelog entry and we can get this released.