Skip to content

Fix charset http Content-Type header#329

Merged
rubyist merged 1 commit intogit-lfs:masterfrom
joerg:fix-http-charset
May 25, 2015
Merged

Fix charset http Content-Type header#329
rubyist merged 1 commit intogit-lfs:masterfrom
joerg:fix-http-charset

Conversation

@joerg
Copy link
Contributor

@joerg joerg commented May 25, 2015

I tried implementing an LFS Server in JAVA and came up with this issue. According to https://tools.ietf.org/html/rfc5987, it has to always be "key = value". I also tried this change and my implementation now works (or at least I am one small step further).

@rubyist
Copy link
Contributor

rubyist commented May 25, 2015

That dash is definitely a typo on our part, thanks for the fix! If you sign the CLA we can merge it in.

rubyist added a commit that referenced this pull request May 25, 2015
Fix charset http Content-Type header
@rubyist rubyist merged commit 8ce6043 into git-lfs:master May 25, 2015
@joerg
Copy link
Contributor Author

joerg commented May 25, 2015

Nice, thanks.
Btw: Read and followed the CONTRIBUTING.md there is no word about the CLA in it. Thats why I did not sign it up front. ;)

@rubyist
Copy link
Contributor

rubyist commented May 25, 2015

That's good feedback, we should definitely mention it in there. I think the only way one finds out about it is the link in a "failing" build after a PR is submitted. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants