-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
move typed headers and implement FromRequest #2094
Conversation
b10e9a1
to
ed33fcc
Compare
4e7d14e
to
ad70715
Compare
31cd96c
to
e6a7cfa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
low langing fruit
not delved in too deep yet, but are the parsing functions (from_one_raw_str etc) still part of -http?
@robjtede I moved the parsing ( |
My thoughts are just that common parsing and formatting functions still belong in -http as long as their inputs or return types are std or HeaderValue or Vec<HeaderValue> etc. |
So which headers from |
i get the impression entity tag is the only one that could move |
…er-from-request
…b into header-from-request
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great. i think this is in a state to get merged after CI passes
Great ! Thank you for implementing this ! |
PR Type
Refactor
PR Checklist
Overview
actix-http
intoactix-web
.Header
extractor.awc::TestResponse
to useIntoHeaderPair
. This is needed now that the typed headers have been moved from-http
.Closes #1837.