-
Notifications
You must be signed in to change notification settings - Fork 315
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
update Rust deps #526
update Rust deps #526
Conversation
set_disposition(&mut response.headers, | ||
xfilename, | ||
headers::Charset::Ext("utf-8".to_string())); | ||
response.headers.set_raw("X-Filename", vec![xfilename.clone().into_bytes()]); |
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.
If you could drop a comment in here so we remember to switch back to set
from set_raw
once it's safe we can get this merged in real quick ;)
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.
done! thanks for the review!
b84019d
to
062559c
Compare
Signed-off-by: Dave Parfitt <[email protected]>
062559c
to
e592eb0
Compare
So many deps, here we go |
📌 Commit e592eb0 has been approved by |
Pull request: #526 Approved by: fnichol
Signed-off-by: Dave Parfitt <[email protected]> Pull request: #526 Approved by: fnichol
☀️ Test successful - travis |
Pull request: #526 Approved by: fnichol
Signed-off-by: Dave Parfitt <[email protected]> Pull request: #526 Approved by: fnichol
Pull request: #526 Approved by: fnichol
using header values created in
depot-core
(Hyper 0.9) doesn't seem to work w/depot
/Iron/Hyper 0.8.I'd be super happy if someone could get this to work without the set_raw hack that I used.
see also: iron/iron#452