bearn01d opened a new pull request, #13816:
URL: https://github.com/apache/apisix/pull/13816

   ### Description
   
   In line with the specification of HTTP/2, APISIX apparently processes all 
headers in lower case. For this reason, entries of the `headers.remove` section 
in `proxy-rewrite` that contain capital letters are ignored, also in cases 
where the client sends them with capital letters. This may lead to unintended 
data leaks.
   This PR transforms headers in `headers.remove` to lower case to ensure 
removal of all headers intended to be removed.
   
   The 
[docs](https://github.com/apache/apisix/blob/master/docs/en/latest/plugins/proxy-rewrite.md#remove-existing-header)
  seem to intend the behavior this PR establishes.
   
   ### Checklist
   
   - [x] I have explained the need for this PR and the problem it solves
   - [x] I have explained the changes or the new features added to this PR
   - [ ] I have added tests corresponding to this change
   - [x] I have updated the documentation to reflect this change
   - [x] I have verified that this change is backward compatible (If not, 
please discuss on the [APISIX mailing 
list](https://github.com/apache/apisix/tree/master#community) first)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to