actix / actix/actix-web

Remove Option from RequestHeadType::Rc

Offen
#2,477 2 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @robjtede Auf GitHub ansehen
A-awc A-http C-improvement
Vorherrschende Sprache
Rust
Sterne
24.8k
Forks
1.9k
Ø Merge
23 Std. 10 Min.
Gemergte PRs (30 T.)
26

Beschreibung

When working with middlewares, I needed to destructure into the ConnectRequest and RequestHeadType to add new headers into the request. If `RequestHeadType::Rc` defaulted to having an empty headermap instead of an Option it would be possible to implement a `headers_mut()` method on ConnectRequest for easy header additions.

I realize that providing a `headers_mut()` for the `RequestHeadType::Rc` case might be misleading, since it won't be possible to remove existing headers, but maybe with a bit of documentation that could be fine

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.