Netty: Ensure PushBuilder headers are added #30771

pnicolucci · 2025-02-11T22:36:23Z

I have considered the risk of behavior change or other zero migration impact (https://github.com/OpenLiberty/open-liberty/wiki/Behavior-Changes).
If this PR fixes an Issue, the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN" (verify release bug label if applicable: https://github.com/OpenLiberty/open-liberty/wiki/Open-Liberty-Conventions).
If this PR resolves an external Known Issue (including APARS), the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN".

pnicolucci · 2025-02-12T12:18:42Z

CTS test that was failing serverPushMiscTest passes with this PR:

pnicolucci added in:Transport team:Sirius labels Feb 11, 2025

pnicolucci self-assigned this Feb 11, 2025

pnicolucci force-pushed the PushBuilderHeaderUpdateNetty branch 4 times, most recently from f724ee2 to 5d5fc18 Compare February 11, 2025 23:06

pnicolucci added 6 commits February 18, 2025 08:47

Clean up class formatting

50753c6

Add PushBuilder headers to Netty Http2Headers

4e790e0

Update debug trace strings to be more concise

21d9497

Add comment to explain why toLowerCase is used

638290a

Make myScheme private and update formatting

d6aeb34

Clean up trace messages and variable names

740e4fe

pnicolucci force-pushed the PushBuilderHeaderUpdateNetty branch from 5d5fc18 to 740e4fe Compare February 18, 2025 13:47

Provide feedback