Fix #[debug_handler] for multiple extractors (#552)

* Fix `#[debug_handler]` for multiple extractors

It generated a function for each extractor to check the type but those
functions didn't have unique names.

Fixed by including all idents in the `arg: Extractor` token tree in the
name of the function generated. Not sure there is a simpler way to fix
it.

* just use a counter

* don't need visit feature anymore

axum-debug/CHANGELOG.md

@@ -7,7 +7,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 # Unreleased
 
-- None.
+- Fix regression causing errors when `#[debug_handler]` was used on functions with multiple
+  extractors.
 
 # 0.2.1 (19. October 2021)
 

axum-debug/src/lib.rs

@@ -206,7 +206,8 @@ mod debug_handler {
             .sig
             .inputs
             .iter()
-            .map(|arg| {
+            .enumerate()
+            .map(|(idx, arg)| {
                 let (span, ty) = match arg {
                     FnArg::Receiver(receiver) => {
                         if receiver.reference.is_some() {
@@ -227,7 +228,11 @@ mod debug_handler {
                     }
                 };
 
-                let name = format_ident!("__axum_debug_check_{}_from_request", item_fn.sig.ident);
+                let name = format_ident!(
+                    "__axum_debug_check_{}_{}_from_request",
+                    item_fn.sig.ident,
+                    idx
+                );
                 quote_spanned! {span=>
                     #[allow(warnings)]
                     fn #name()

axum-debug/tests/pass/multiple_extractors.rs

@@ -0,0 +1,6 @@
+use axum_debug::debug_handler;
+
+#[debug_handler]
+async fn handler(_one: String, _two: String, _three: String) {}
+
+fn main() {}