actix-web
actix-web copied to clipboard
remove http::Uri from public api
http::Uri
is fine for transport layer, but it is pretty crappy as public api
Do you intend to replace it with something?
We still need some Uri manipulation options, even crappy http:Uri
is fine
we have to replace it with something.
i started work on new functionality.
@DoumanAsh i am going to deprecate .uri_mut()
,
you can not modify uri for path normalization, routing happens before any middleware get called.
for normalizatin we need to come up with different approach. i am thinking about sub-request.
Some way to get the full URL from a request would definitely be useful.
@fafhrd91 Do we still want to remove http::Uri
from publci API?
We need replacement first
BTW, WHATWG recommends using the term URL, because the new spec has unified or replaced the old theoretical flavors of {U,I}R{I,L}s.
https://url.spec.whatwg.org/#goals
So while you're changing it, I recommend switching to the name URL. It may even help with the transition.
And there's the url crate based on the current standard.