ashakirin commented on a change in pull request #681:
URL: https://github.com/apache/cxf/pull/681#discussion_r456093246
##########
File path:
rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java
##########
@@ -95,6 +97,17 @@ public LogEvent map(Message message) {
return event;
}
+ private Map<String, String> maskHeaders(
+ final Map<String, String> headerMap,
+ final Map<String, Boolean> sensitiveHeaderMap) {
+ final Map<String, String> maskedHeaderMap = new HashMap<>();
+ headerMap.keySet().forEach(key -> {
+ maskedHeaderMap.put(key, sensitiveHeaderMap.containsKey(key)
+ ? MASKED_HEADER_VALUE : headerMap.get(key));
+ });
Review comment:
Have replaced List on Set for all arguments, makes more sense and it is
performant
----------------------------------------------------------------
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]
