fix(rest-connector): Return always an array for set cookie headers #4202
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.
Description
Restore the default behavior of 8.5. Set cookie headers a always merged into an array. The flag for grouping is removed. This also means, if someone sets the group flag to false, this will be ignored after an update, but I think that is ok, as this is not really a helpful feature and I guess noone will do this.
This will be documented here
I did not document the "breaking change" in the last two lines of the table, because i consider this a bug fix and not a breaking change. Let me know if you disagree, then I will add it to the docs. (We could adjust the behavior so that if there is only one Set-Cookie header, it returns a simple string instead of an array. However, this approach is not ideal. For example, if an endpoint sometimes returns one cookie (e.g., when login fails) and sometimes two (e.g., when login succeeds), it would make handling cookies in FEEL more complex.)
Related issues
closes #4201
Checklist
no milestone
label.