feat: support String-Array query parameter in DP public API #4051
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR changes/adds
This PR adds support for multiple identical HTTP query parameters. For example, parameters like
foo=bar&foo=baz
were collapsed, and onlyfoo=bar
was forwarded to the HTTP target.With this PR, the same query param string is preserved as string-array.
Why it does that
A Tractus-X system standard requires the support of string-array query parameters.
Further notes
DataPlanePublicApiEndToEndTest
was improved to verify HTTP requests using the Netty MockServer (instead of calling a dummy URL)DataPlanePublicApiV2
now supportsHEAD
requests, they were missing before.PULL
flowsNB: that this behaviour will NOT be backported to the legacy version of the public dataplane API!
Linked Issue(s)
Closes #4022
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.