moby / moby/spdystream

Configurable header compression

Open
#42 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Go
Stars
149
Forks
59
PR merge metrics
No merged PRs in 30d

Description

Currently the spdy framer uses the best compression setting for zlib, which is also the slowest. The compression should be configurable on connection creation to allow for less overhead during spdy stream creation. In libchan profiling switching from BestCompression to BestSpeed reduced the time overhead of compression from 24% to 4% for the same test. Since spdystream is designed to be used over any type of net.Conn, this should be configurable to match the net.Conn speed capabilities.

Changing this will involve vendoring and altering the go.net spdy package. However the spdy package appears to be frozen so I don't think that is a problem.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the connection-creation entry point and the vendored go.net spdy package, then review how the framer selects zlib compression. Use the libchan profiling comparison as context. Done means callers can choose the compression setting when creating a connection without losing the current default behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
networking
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.