hrw4u: u4wrh emits outbound.req.* into READ_REQUEST/PRE_REMAP, which hrw4u rejects
Maintainers usually reply within 2 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- python
- Domain
- networking
Research direction
Start with the section mappings in src/generators.py:70 and src/tables.py:52, then run the shown HRW input through u4wrh and hrw4u. Check the set-header, rm-header, and add-header cases in PRE_REMAP and READ_REQUEST, while preserving the valid SEND_REQUEST behavior; done means hrw4u accepts the generated directives in the affected sections.
Written by the indexing model from the issue text.
Description
Found while reviewing #13204 — reproduces on master.
Input HRW:
cond %{READ_REQUEST_HDR_HOOK}
set-header X-Foo "bar"
What u4wrh produces:
READ_REQUEST {
outbound.req.X-Foo = "bar";
}
Which hrw4u then rejects:
error: outbound.req.X-Foo is not available in the READ_REQUEST section
The reverse mapping and the section table disagree: HEADER_CONTEXT_MAP maps PRE_REMAP/READ_REQUEST/SEND_REQUEST to outbound.req. (src/generators.py:70), while OPERATOR_MAP["outbound.req."] declares sections={SEND_REQUEST, READ_RESPONSE, SEND_RESPONSE} (src/tables.py:52). SEND_REQUEST is fine; READ_REQUEST and PRE_REMAP are not.
Affects set-header, rm-header and add-header in those two sections.
- Dominant language
- C++
- Stars
- 2k
- Forks
- 878
- Avg merge
- 3d 13h
- Merged PRs (30d)
- 86
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing 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.
More from apache/trafficserver
-
hrw4u: grammar accepts bare operator statements (`no-op;`) that can never compilePossibly taken @masaori335 claimed this 11 days ago. Openhrw4u
Difficulty 1/5 Under an hour Newbie friendliness 88/100
apache/trafficserver#13701 · 1 assignee ·
Maintainers usually reply within 2 days
-
Bug HTTP Support
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
apache/trafficserver#13118 ·
Maintainers usually reply within 2 days
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
apache/trafficserver#13784 ·
Maintainers usually reply within 2 days
-
HTTP/2
Difficulty 4/5 3-5 days Newbie friendliness 68/100
apache/trafficserver#13778 ·
Maintainers usually reply within 2 days
-
Plugins
Difficulty 5/5 Over a week Newbie friendliness 35/100
apache/trafficserver#13774 ·
Maintainers usually reply within 2 days
All issues in apache/trafficserver
Similar issues
-
upstream update
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
conan-io/conan-center-index#31098 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
ml-explore/mlx-c#136 ·
-
bug chart-audit
Difficulty 1/5 Under an hour Newbie friendliness 92/100
Maintainers usually reply within 1 day