Skip to content
This repository has been archived by the owner on Jul 28, 2020. It is now read-only.

Fix: HttpConnectProxy partial support for Titanium/Apache, and add tests. #105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kenjiuno
Copy link
Contributor

Fix: HttpConnectProxy partial support for Titanium/Apache.

  • This fix supports the following pattern, receiving 2 chunks:
    HTTP/1.1 200 Connection Established\r\n and
    content-length: 0\r\n\r\n
    It is covered by test TestHttp11ComplexMultiPacket
  • This fix does not support the following pattern. In case of: proxy server sends both http response header and body in one packet!
    HTTP/1.1 403 Forbidden\r\n ... </body></html>
    This failure test is covered by test TestHttp11ComplexOnePacketAsForbidden

Fix #104

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.

Fix: HttpConnectProxy partial support for Titanium/Apache, and add tests
1 participant