Add HTTP/2 Frame Flags support
Open
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 47
- Forks
- 29
- Avg merge
- 5d 6h
- Merged PRs (30d)
- 3
Description
Feature request: it'd great if HTTP/2 frame flags can be configured to test edge cases,
- e.g.
transactions:
- client-request:
frames:
- HEADERS:
flags:
END_HEADERS: true
END_STREAM: false
- DATA:
flags:
END_STREAM: true
For now, it's automatically set
https://github.com/yahoo/proxy-verifier/blob/master/local/src/core/http2.cc#L1510-L1520
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in local/src/core/http2.cc around lines 1510-1520, where frame flags are currently set automatically. Trace how the YAML transaction frame definitions are parsed, then support configurable HEADERS and DATA flags like those shown in the issue. Done means the requested flags can be specified per frame instead of always being selected automatically.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- networking
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100