Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"http_server" input: query parameter overwrites header field with the same name #136

Open
gramian opened this issue Nov 21, 2024 · 0 comments
Labels
enhancement New feature or request http Any tasks or issues relating specifically to http components metadata Any tasks or issues relating specifically to metadata handling

Comments

@gramian
Copy link
Contributor

gramian commented Nov 21, 2024

When using the http_server input, the query parameters and header fields are both kept in the metadata. This has the consequence that if a query parameter has the name as a header field, the header field value gets overwritten with the query parameter. This likely hasn't been reported due to the odd chances of query parameters and header field names coinciding (also due to differing capitalization traditions).

As @mihaitodor suggested on Discord, where I asked about this behavior, that extra parameters could be introduced to configure different namespaces (i.e. sub-objects of meta) for query parameters and header fields.

I like this idea as it would also enable to test for request errors 414 (Request URI too long) and 431 (header fields too large) correctly, which is currently not possible.

@mihaitodor mihaitodor added enhancement New feature or request http Any tasks or issues relating specifically to http components metadata Any tasks or issues relating specifically to metadata handling labels Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request http Any tasks or issues relating specifically to http components metadata Any tasks or issues relating specifically to metadata handling
Projects
None yet
Development

No branches or pull requests

2 participants