Setup Content-Type to application/octet-stream by default

[avamsi]

Related issue number

#944

Checklist

• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes

[avamsi]

@asvetlov Just saw the failing tests. Sorry about that.
If you can take a look at the code changes and confirm everything's okay, I'll update affected tests accordingly.

[asvetlov]

Please use " for exception messages

[avamsi]

I thought the difference in quotes usage was due to different contributors. I'll change it back

[asvetlov]

Please don't forget about:

I think this code should be in StreamResponse ctor.
Content-Type should be application/octet-stream if not explicitly specified.
It's true for all responses.

See https://tools.ietf.org/html/rfc7231#section-3.1.1.5

[codecov-io]

Current coverage is 98.15% (diff: 100%)

Merging #1124 into master will increase coverage by <.01%

@@             master      #1124   diff @@
==========================================
  Files            28         28          
  Lines          6387       6391     +4   
  Methods           0          0          
  Messages          0          0          
  Branches       1073       1074     +1   
==========================================
+ Hits           6269       6273     +4   
  Misses           63         63          
  Partials         55         55          

Powered by Codecov. Last update 19c0f31...ce207aa

[asvetlov]

Thanks!

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.