actix/actix-web
1
0
Fork 0
mirror of https://github.com/fafhrd91/actix-web synced 2024-11-30 10:42:55 +01:00
Code Issues Releases Wiki Activity

match HttpRequest app_data behavior in ServiceRequest (#1618)

Browse Source
This commit is contained in:
Rob Ede 2020-08-09 15:51:38 +01:00 committed by GitHub
parent 46627be36f
commit 187646b2f9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 29 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
CHANGES.md
View File

@ -6,6 +6,8 @@
using `App::data`. [#1610] using `App::data`. [#1610]
* `web::Path` now has a public representation: `web::Path(pub T)` that enables * `web::Path` now has a public representation: `web::Path(pub T)` that enables
destructuring. [#1594] destructuring. [#1594]
* `ServiceRequest::app_data` allows retrieval of non-Data data without splitting into parts to
access `HttpRequest` which already allows this. [#1618]
* MSRV is now 1.42.0. * MSRV is now 1.42.0.
### Fixed ### Fixed
@ -14,6 +16,7 @@
[#1594]: https://github.com/actix/actix-web/pull/1594 [#1594]: https://github.com/actix/actix-web/pull/1594
[#1609]: https://github.com/actix/actix-web/pull/1609 [#1609]: https://github.com/actix/actix-web/pull/1609
[#1610]: https://github.com/actix/actix-web/pull/1610 [#1610]: https://github.com/actix/actix-web/pull/1610
[#1618]: https://github.com/actix/actix-web/pull/1610
## 3.0.0-beta.1 - 2020-07-13 ## 3.0.0-beta.1 - 2020-07-13

32
src/service.rs
View File

@ -12,7 +12,6 @@ use actix_router::{IntoPattern, Path, Resource, ResourceDef, Url};
use actix_service::{IntoServiceFactory, ServiceFactory}; use actix_service::{IntoServiceFactory, ServiceFactory};
use crate::config::{AppConfig, AppService}; use crate::config::{AppConfig, AppService};
use crate::data::Data;
use crate::dev::insert_slash; use crate::dev::insert_slash;
use crate::guard::Guard; use crate::guard::Guard;
use crate::info::ConnectionInfo; use crate::info::ConnectionInfo;
@ -226,12 +225,11 @@ impl ServiceRequest {
self.0.app_config() self.0.app_config()
} }
/// Get an application data stored with `App::data()` method during /// Counterpart to [`HttpRequest::app_data`](../struct.HttpRequest.html#method.app_data).
/// application configuration. pub fn app_data<T: 'static>(&self) -> Option<&T> {
pub fn app_data<T: 'static>(&self) -> Option<Data<T>> {
for container in (self.0).0.app_data.iter().rev() { for container in (self.0).0.app_data.iter().rev() {
if let Some(data) = container.get::<Data<T>>() { if let Some(data) = container.get::<T>() {
return Some(Data::clone(&data)); return Some(data);
} }
} }
@ -595,6 +593,28 @@ mod tests {
let resp = srv.call(req).await.unwrap(); let resp = srv.call(req).await.unwrap();
assert_eq!(resp.status(), http::StatusCode::NOT_FOUND); assert_eq!(resp.status(), http::StatusCode::NOT_FOUND);
} }
#[actix_rt::test]
async fn test_service_data() {
let mut srv = init_service(
App::new()
.data(42u32)
.service(web::service("/test").name("test").finish(
|req: ServiceRequest| {
assert_eq!(
req.app_data::<web::Data<u32>>().unwrap().as_ref(),
&42
);
ok(req.into_response(HttpResponse::Ok().finish()))
},
)),
)
.await;
let req = TestRequest::with_uri("/test").to_request();
let resp = srv.call(req).await.unwrap();
assert_eq!(resp.status(), http::StatusCode::OK);
}
#[test] #[test]
fn test_fmt_debug() { fn test_fmt_debug() {
let req = TestRequest::get() let req = TestRequest::get()