Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Added some ietf rfc links and multi file test #20

Closed
wants to merge 3 commits into from
Closed

Added some ietf rfc links and multi file test #20

wants to merge 3 commits into from

Conversation

psdh
Copy link
Contributor

@psdh psdh commented Dec 21, 2015

@@ -14,6 +14,7 @@ use hyper::header::{Header, HeaderFormat, parsing};
///
/// This is an internal type not meant for general public usage, as its implementation limited in

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There still might be value in upstreaming this to hyper since servers may have to parse header this as well. cc @seanmonstar

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've already upstreamed a better version of the ContentDisposition header, but clearly I never updated formdata to use it. I'll get onto that.

@mikedilger
Copy link
Owner

Thanks for this. Master has diverged. Please rebase onto master.

By the way, recent changes on master broke formdata because of a serde_codegen bug serde-rs/serde#196 but I've fixed that by committing the generated code. So master should be OK again.

@mikedilger
Copy link
Owner

I have just rebased and merged

@mikedilger mikedilger closed this Dec 21, 2015
@psdh
Copy link
Contributor Author

psdh commented Dec 22, 2015

@mikedilger what is a good way to contact you? IRC? the mail you have listed in your github profile seems to bouncing back.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants