Skip to content

echo: add echo server #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

echo: add echo server #11

wants to merge 10 commits into from

Conversation

Spiffyk
Copy link

@Spiffyk Spiffyk commented Jul 31, 2023

This commit adds an echo server that reflects queries back at the sender. May be used for determining the baseline performance of QUIC (or, at least, of the quic-go implementation in combination with the client's QUIC implementation).

fcelda
fcelda requested changes Aug 30, 2023
View reviewed changes
Copy link
Member

@fcelda fcelda left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution. Please, do you mind checking the comments below? I don't insist on all changes.

@Spiffyk
echo: add echo server
492421d 
This commit adds an echo server that reflects queries back at the
sender. May be used for determining the baseline performance of QUIC
(or, at least, of the `quic-go` implementation in combination with the
client's QUIC implementation).
@Spiffyk
echo: replace io.ReadAll with stream.Read
c644a64 
ReadAll is apparently SLOW for our purposes. Reading into a fixed-size
buffer seems to be much better.
@Spiffyk
Exclude go.sum
7c764f9
@Spiffyk
server: code deduplication
d0861f9
@Spiffyk
Update quic-go, enable 0-RTT, more error logging
c5b094e
@Spiffyk
Restore the limit on maximum concurrent streams
fcc76c7
@Spiffyk
echo: QR-bit-setting DNS mode
e63ec77
@Spiffyk
Refactor and deduplicate server parameters using generics
70f8e5a
@Spiffyk
server: add option to specify TLS key log file
b681397 
This allows developers to analyze the traffic using Wireshark, even when
it is encrypted.
@Spiffyk
Copy link
Author

Spiffyk commented Dec 7, 2023

I have updated quic-go, which now supports 0-RTT (supported by the in-development version of DNS Shotgun with QUIC support - I use this echo server to test that). It now however requires Go >=1.20 - is that okay?

Another improvement is the addition of the -keylog string flag that makes the server (both echo and proxy) write TLS keys into the specified file, which is useful for Wireshark analysis.

@Spiffyk Spiffyk requested a review from fcelda December 7, 2023 16:08
@Spiffyk
server: random stream reset option (opt-in)
2272da0 
For testing purposes, a -reset option that randomly resets streams.
@Spiffyk Spiffyk mentioned this pull request Dec 20, 2023
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fcelda fcelda Awaiting requested review from fcelda

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Spiffyk @fcelda