Skip to content

FIX: update the format of response based on rfc2616#2140

Merged
sswguo merged 1 commit intoCommonjava:2.7.xfrom
sswguo:httprox_fix
Aug 26, 2022
Merged

FIX: update the format of response based on rfc2616#2140
sswguo merged 1 commit intoCommonjava:2.7.xfrom
sswguo:httprox_fix

Conversation

@sswguo
Copy link
Copy Markdown
Contributor

@sswguo sswguo commented Aug 26, 2022

Looks .net client expects an extra CRLF in the response following rfc2616 .
generic-message = start-line *(message-header CRLF) CRLF [ message-body ] start-line = Request-Line | Status-Line
I had test this in generic proxy service, and this is added for production, since we still have not switch proxy on prod. thanks.

@ruhan1
Copy link
Copy Markdown
Contributor

ruhan1 commented Aug 26, 2022

LGTM

@sswguo sswguo requested review from ligangty and ruhan1 August 26, 2022 05:45
Copy link
Copy Markdown
Contributor

@ligangty ligangty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sswguo sswguo merged commit 433a31c into Commonjava:2.7.x Aug 26, 2022
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.

3 participants