mirror of https://github.com/fafhrd91/actix-web
59244b203c
A very minor addition: I'm using this middleware on specific resources,
and given a non-static string, I often have to `clone()` already to get
a string into a closure. Take this code for example:
``` rust
let server = actix_web::server::new(move || {
let csrf_origin_graphql = csrf_origin.clone();
...
.resource("/graphql", move |r| {
r.middleware(
csrf::CsrfFilter::new().allowed_origin(csrf_origin_graphql.as_str()),
);
r.method(Method::POST).a(graphql::handlers::graphql_post);
})
```
Letting `allowed_origin()` take an `Into<String>` instead of `&str` would
prevent a second `clone()` in the code above, and also make the code a little
nicer to read (you eliminate the `.as_str()` above). This is a pattern that
seems to be common throughout actix-web already anyway, so it should also be
fine to have here.
|
||
|---|---|---|
| .. | ||
| client | ||
| header | ||
| middleware | ||
| server | ||
| ws | ||
| application.rs | ||
| body.rs | ||
| context.rs | ||
| de.rs | ||
| error.rs | ||
| extractor.rs | ||
| fs.rs | ||
| handler.rs | ||
| helpers.rs | ||
| httpcodes.rs | ||
| httpmessage.rs | ||
| httprequest.rs | ||
| httpresponse.rs | ||
| info.rs | ||
| json.rs | ||
| lib.rs | ||
| multipart.rs | ||
| param.rs | ||
| payload.rs | ||
| pipeline.rs | ||
| pred.rs | ||
| resource.rs | ||
| route.rs | ||
| router.rs | ||
| test.rs | ||
| uri.rs | ||
| with.rs | ||