Skip to content
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.

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

Use HTTP compressed encoding when available #143

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Conversation

job
Copy link
Contributor

@job job commented Aug 3, 2024

This reduces network traffic by about 50%.

Without patch applied:

# systemd-run -p IPAccounting=yes --wait ./src/fort --tal=/etc/tals/ --mode=standalone --output.roa=/tmp/roa1 --local-repository=/home/fort1
Running as unit: run-u5244.service
Finished with result: success
Main processes terminated with: code=exited/status=0
Service runtime: 9min 243ms
CPU time consumed: 6min 59.190s
IP traffic received: 1.1G
IP traffic sent: 17.5M

With patch applied:

# systemd-run -p IPAccounting=yes --wait ./src/fort --tal=/etc/tals/ --mode=standalone --output.roa=/tmp/roa2 --local-repository=/home/fort-http-compression
Running as unit: run-u5246.service
Finished with result: success
Main processes terminated with: code=exited/status=0
Service runtime: 8min 44.294s
CPU time consumed: 10min 44.351s
IP traffic received: 572.1M
IP traffic sent: 8.3M

Comparison with rpki-client which uses HTTP compression when available:

# systemd-run -p IPAccounting=yes --wait rpki-client -v -d /home/rpkiclient /tmp
Running as unit: run-u5208.service
Finished with result: success
Main processes terminated with: code=exited/status=0
Service runtime: 7min 17.549s
CPU time consumed: 5min 27.936s
IP traffic received: 547.3M
IP traffic sent: 8.6M

This reduces network traffic by about 50%.
@ydahhrk ydahhrk merged commit d8e36c5 into NICMx:main Aug 6, 2024
@ydahhrk
Copy link
Member

ydahhrk commented Aug 6, 2024

Thanks; this is awesome!

@ydahhrk ydahhrk added this to the 1.6.3 milestone Aug 26, 2024
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.

2 participants