Bug 60573 - Drop reason phrase of HTTP/1.1 100 (Expect: 100-continue)
Summary: Drop reason phrase of HTTP/1.1 100 (Expect: 100-continue)
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 8
Classification: Unclassified
Component: Catalina (show other bugs)
Version: 8.5.x-trunk
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ----
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-11 12:08 UTC by Michael Osipov
Modified: 2017-03-06 12:59 UTC (History)
0 users



Attachments
Drop reason phrase (1.46 KB, patch)
2017-01-11 12:08 UTC, Michael Osipov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Osipov 2017-01-11 12:08:20 UTC
Created attachment 34609 [details]
Drop reason phrase

Tomcat 8.5 dropped optional reason phrases but HTTP/1.1 100 still sends it. Having all status code to be consistent, I have aligned Http11OutputBuffer's #sendAck() to #sendStatus(), dropping "Continue" on the Constants#ACK_BYTES. Also updated TestByteChunk which obviously validated HTTP/1.1 100 status line byte chunk.
Comment 1 Christopher Schultz 2017-01-11 22:43:45 UTC
Note to committers: please do not back-port this change to 8.0.x and earlier. I'd like to raise the issue of restoring the reason phrase to 8.0.x.
Comment 2 Michael Osipov 2017-01-12 08:35:14 UTC
(In reply to Christopher Schultz from comment #1)
> Note to committers: please do not back-port this change to 8.0.x and
> earlier. I'd like to raise the issue of restoring the reason phrase to 8.0.x.

I expect this for 8.5 and 9.0 only.
Comment 3 Michael Osipov 2017-01-20 13:30:39 UTC
Anyone willing to take a look at this trivial patch?
Comment 4 Michael Osipov 2017-02-10 09:54:46 UTC
Committers, this is trivial.
Comment 5 Mark Thomas 2017-03-06 12:35:56 UTC
Fixed in 9.0.x for 9.0.0.M18 onwards.

8.0.x has always sent the reason phrase.

It is 8.5.x where there is an open enhancement request to (optionally) restore the reason phrase. On the basis that that enhancement request will be implemented, I'm not planning on back-porting this fix to 8.5.x.
Comment 6 Michael Osipov 2017-03-06 12:59:59 UTC
(In reply to Mark Thomas from comment #5)
> Fixed in 9.0.x for 9.0.0.M18 onwards.
> 
> 8.0.x has always sent the reason phrase.
> 
> It is 8.5.x where there is an open enhancement request to (optionally)
> restore the reason phrase. On the basis that that enhancement request will
> be implemented, I'm not planning on back-porting this fix to 8.5.x.

I am are of the fact. I have created this issue for consistency. I personally do not care about the reason phrase. The mentioned firmwares on those embedded devices are just broken/non-compliant.