rwf2/Rocket

Phase out async_trait macro

Open

#2684 aperta il 30 dic 2023

Vedi su GitHub
 (5 commenti) (6 reazioni) (0 assegnatari)Rust (1645 fork)batch import
acceptedhelp wantedrequest

Metriche repository

Star
 (25.738 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

What's missing?

Rocket currently re-exports the async_trait crate for use with its Fairing (and possibly others). As of rust 1.75.0 this is no longer needed, therefore it would be nice to get rid of this dependency and use actual async functions for these traits.

This generally improves compile times, dependency count and IDE responsiveness.

Ideal Solution

For v0.6, remove async_trait (or put its usage behind a feature) and use async functions directly. This would raise the MSRV to 1.75.0.

Why can't this be implemented outside of Rocket?

This is an integral part of Rocket.

Are there workarounds usable today?

No. You bypass the need for the macro attribute by returning a Box<dyn Future<Output = T> + Send + 'a>, but this is unwieldy.

Alternative Solutions

No response

Additional Context

No response

System Checks

  • I do not believe that this feature can or should be implemented outside of Rocket.
  • I was unable to find a previous request for this feature.

Guida contributor

Direzione di ricerca
Cerca tutti gli utilizzi di `async trait` nella codebase di Rocket, in particolare nel trait `Fairing` e in qualsiasi altro trait che attualmente utilizza l'attributo `#[async trait]`. Controlla `Cargo.toml` per la dipendenza `async trait` e i suoi flag di funzionalità. L'obiettivo è sostituire ogni attributo `#[async trait]` con `async fn` nelle definizioni e implementazioni dei trait, e rimuovere la dipendenza. Considera di alzare l'MSRV a 1.75.0 come menzionato.
Tech stack
rust
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60