elasticsearch-js icon indicating copy to clipboard operation
elasticsearch-js copied to clipboard

fix(transport): merge default headers with custom headers (#2862)

Open jerinthomascarmel opened this issue 5 months ago • 2 comments

Fix header merge logic to preserve default headers when custom headers are provided

Changes: Update header-merge logic to explicitly spread default headers and then apply opts.headers, ensuring defaults are preserved.

Closes: #2862

Testing: Run npm test to ensure the new header-merge test passes.

jerinthomascarmel avatar Jun 15 '25 17:06 jerinthomascarmel

❌ Author of the following commits did not sign a Contributor Agreement: 1dc54bb2a42f17c877266b5f452bb0a9aa026dca

Please, read and sign the above mentioned agreement if you want to contribute to this project

:tada: Snyk checks have passed. No issues have been found so far.

:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)

:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)

prodsecmachine avatar Jun 15 '25 17:06 prodsecmachine

Closing in favor of #2890 which has unit tests, solves for more than one case, and does not need to wait for a CLA agreement to be signed.

JoshMock avatar Jul 02 '25 21:07 JoshMock