[GitHub] [tomcat] markt-asf commented on pull request #550: Http11Processor's keep alive state and input buffer's swallow state must be synchronized

markt-asf commented on PR #550: URL: https://github.com/apache/tomcat/pull/550#issuecomment-1240278052 > Are you referring to the call to `inputBuffer.setSwallowInput(true);` in `Http11Processor.ack`? That does appear to be unnecessary now. Yes. That was the one. -- This is an auto

[GitHub] [tomcat] markt-asf commented on pull request #550: Http11Processor's keep alive state and input buffer's swallow state must be synchronized

markt-asf commented on PR #550: URL: https://github.com/apache/tomcat/pull/550#issuecomment-1239284952 I suspect the call in `Http11Processor.ack()` can also be removed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [tomcat] markt-asf commented on pull request #550: Http11Processor's keep alive state and input buffer's swallow state must be synchronized

markt-asf commented on PR #550: URL: https://github.com/apache/tomcat/pull/550#issuecomment-1238513982 I think you can just remove the `inputBuffer.setSwallowInput(false)` in `Http11Processor.prepareExpectation()`. If that is correct, I think there is at least one other call that can also b