moresandeep commented on a change in pull request #423:
URL: https://github.com/apache/knox/pull/423#discussion_r616077596
##########
File path:
gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/ConfigurableDispatch.java
##########
@@ -41,13 +47,26 @@
public class ConfigurableDispatch extends DefaultDispatch {
private Set<String> requestExcludeHeaders =
super.getOutboundRequestExcludeHeaders();
private Set<String> responseExcludeHeaders =
super.getOutboundResponseExcludeHeaders();
+ private Map<String, String> requestAppendHeaders = new HashMap<>();
private Set<String> responseExcludeSetCookieHeaderDirectives =
super.getOutboundResponseExcludedSetCookieHeaderDirectives();
private Boolean removeUrlEncoding = false;
private Set<String> convertCommaDelimitedHeadersToSet(String headers) {
return headers == null ? Collections.emptySet(): new
HashSet<>(Arrays.asList(headers.split("\\s*,\\s*")));
}
+ private Map<String, String> convertJSONHeadersToMap(String headers) {
Review comment:
Apologies for the delay, the simple param example looks good to me, I
cannot think of a case where we need complex param, do you have one in mind?
AFA headers are concerned they are simple <key><value> so I do not think we
need to complicate this, now depending on header the separator might vary (e.g.
SET-COOKIE header) but that should not be a problem here. We could do a simple
validation to make sure that the header values don't break the request.
Test would be great to have we don't this to break accidentally due to any
later change.
Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
