Data::<T>::into_inner requires T: Sized unnecessarily

[jarhodes314]

Expected Behavior

I have some code that currently looks something like:

trait MyTrait {
   // ...
}

#[post("/some-path")]
pub async fn do_something(data: Data<dyn MyTrait>) {
    Container(data.into_inner()).do_something();
}

struct Container(Arc<dyn MyTrait>);

impl Container {
   // ...
}

However, this fails to compile since into_inner is only implemented for Data<T> (which has an implicit Sized bound on T).

Current Behavior

The above example doesn't compile since into_inner is only implemented for Data<T> (which has an implicit Sized bound on T). Instead, the handler has to call:

Container((*data).clone()).do_something()

to retrieve an owned Arc<dyn MyTrait>. This is both harder to read and unnecessary cloning.

Possible Solution

into_inner should be in an impl Data<T: ?Sized> block instead of impl Data.