Open iampopovich opened 3 weeks ago
β±οΈ Estimated effort to review [1-5] | 2, because the changes are limited to a few lines in a single file, modifying header handling to be case-insensitive. The logic is straightforward, but the impact on existing functionality should be carefully evaluated. |
π§ͺ Relevant tests | No |
β‘ Possible issues | Consistency Issue: The PR changes headers to be stored in lowercase, which might affect other parts of the system that rely on the original case format of headers. |
π Security concerns | No |
Category | Suggestion | Score |
Possible bug |
Add a null check for the
___
**Consider adding a null check for the | 8 |
Best practice |
Use
___
**Consider using | 7 |
Let's at least wait to merge this after the upcoming release so that Appium can run their tests against nightly and see if it causes them problems before a production release.
User description
Thanks for contributing to Selenium! A PR well described will help maintainers to quickly review and merge it
Before submitting your PR, please check our contributing guidelines. Avoid large PRs, help reviewers by making them as simple and short as possible.
Description
I opened PR with initial changes. i have already asked some questions about feature in feature request ticket I will implement new tests as soon as I get more information about task details
Motivation and Context
according to #12697 I started to discover how to implement this feature
Types of changes
Checklist
PR Type
Enhancement
Description
setHeader
method to ensure case-insensitive handling.addHeader
method to ensure case-insensitive handling.Changes walkthrough π
HttpMessage.java
Normalize header names to lowercase in HTTP message handling.
java/src/org/openqa/selenium/remote/http/HttpMessage.java
setHeader
method.addHeader
method.