Repository navigation
🐛 Fix authority header handling for HTTP/2 - #2522
Conversation
Signed-off-by: Alex Li <[email protected]>
|
@copilot Add authority tests in the dio_test package |
Implemented in commit I added shared authority regression coverage in |
|
@copilot The test should not only targeting H2, but as a general test about header values |
Updated in I refactored this into a general shared header-value test under |
|
@copilot Can't the test check the value directly without determining the adapter? If not, convert the test from general to the H2 adapter package only (and move under its test folder) |
Updated in I moved this regression test to |
|
Addressed in I removed the unnecessary |
|
@copilot Format the test file |
Done in |
|
@copilot Add changelog entry |
Done in |
Code Coverage Report: Only Changed Files listed
Minimum allowed coverage is |
Fixes #2521