actix/actix-web
1
0
Fork 0
mirror of https://github.com/fafhrd91/actix-web synced 2024-11-23 16:21:06 +01:00
Code Issues Releases Wiki Activity

Add ability to name a handler function as 'config' (#1290)

Browse Source 
* eliminate handler naming restrictions #1277

* Update actix-web-codegen/CHANGES.md

Co-authored-by: Yuki Okushi <huyuumi.dev@gmail.com>
This commit is contained in:
Andrey Torsunov 2020-01-26 01:22:40 +03:00 committed by Yuki Okushi
parent 8888520d83
commit 71d11644a7
3 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
actix-web-codegen/CHANGES.md
View File

@ -1,5 +1,9 @@
# Changes
## [0.2.NEXT] - 2020-xx-xx
* Allow the handler function to be named as `config` #1290
## [0.2.0] - 2019-12-13
* Generate code for actix-web 2.0

6
actix-web-codegen/src/route.rs
View File

@ -195,15 +195,15 @@ impl Route {
pub struct #name;
impl actix_web::dev::HttpServiceFactory for #name {
fn register(self, config: &mut actix_web::dev::AppService) {
fn register(self, __config: &mut actix_web::dev::AppService) {
#ast
let resource = actix_web::Resource::new(#path)
let __resource = actix_web::Resource::new(#path)
.name(#resource_name)
.guard(actix_web::guard::#guard())
#(.guard(actix_web::guard::fn_guard(#extra_guards)))*
.#resource_type(#name);
actix_web::dev::HttpServiceFactory::register(resource, config)
actix_web::dev::HttpServiceFactory::register(__resource, __config)
}
}
};

6
actix-web-codegen/tests/test_macro.rs
View File

@ -2,6 +2,12 @@ use actix_web::{http, test, web::Path, App, HttpResponse, Responder};
use actix_web_codegen::{connect, delete, get, head, options, patch, post, put, trace};
use futures::{future, Future};
// Make sure that we can name function as 'config'
#[get("/config")]
async fn config() -> impl Responder {
HttpResponse::Ok()
}
#[get("/test")]
async fn test_handler() -> impl Responder {
HttpResponse::Ok()