Skip to content

Comments

Buffer only proxy header data#119

Closed
AlexanderYastrebov wants to merge 4 commits intopires:mainfrom
AlexanderYastrebov:no-buffer
Closed

Buffer only proxy header data#119
AlexanderYastrebov wants to merge 4 commits intopires:mainfrom
AlexanderYastrebov:no-buffer

Conversation

@AlexanderYastrebov
Copy link

@AlexanderYastrebov AlexanderYastrebov commented Oct 10, 2024

Reverts and re-implements incorrect #116
which passed all connection data through the buffered reader.

@pires
Copy link
Owner

pires commented Oct 11, 2024

@mmatczuk care to take a look at this?

@coveralls
Copy link

coveralls commented Oct 11, 2024

Coverage Status

coverage: 94.67% (+0.5%) from 94.136%
when pulling 0067a88 on AlexanderYastrebov:no-buffer
into bac82fd on pires:main.

Reverts and re-implements incorrect pires#116
which passed all connection data through the buffered reader.
@mmatczuk
Copy link
Contributor

I added my comments in #116 (comment).

@mmatczuk
Copy link
Contributor

Sent a followup patch #120.

@AlexanderYastrebov

This comment was marked as outdated.

@mmatczuk
Copy link
Contributor

@AlexanderYastrebov

It also has issues.

PTAL at my comments in #120.

Besides I fixed WriteTo instead of hand-rolled reads.

Sure, this is also fixed in #120

BTW adding the WriteTo to the interface if the underlying connection does not implement in may trick io.Copy with buffer from a buffer pool to falling back to plain io.Copy that allocates. Also the WriteTo only works on Linux IF the writer is UnixConn.

Cheers

@pires
Copy link
Owner

pires commented Feb 6, 2026

Fixed in #148

@pires pires closed this Feb 6, 2026
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.

4 participants