anton-vinogradov commented on a change in pull request #9814:
URL: https://github.com/apache/ignite/pull/9814#discussion_r816975945
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/consistency/GridNearReadRepairAbstractFuture.java
##########
@@ -388,6 +398,58 @@ protected final void recordConsistencyViolation(
strategy));
}
+ /**
+ *
+ */
+ private Object sensitiveValue(boolean includeSensitive, EntryGetResult res,
+ Map<ByteArrayWrapper, Object> sensitiveValMap) {
+ if (res != null) {
+ CacheObject val = res.value();
+
+ try {
+ ByteArrayWrapper wrapped = new
ByteArrayWrapper(val.valueBytes(ctx.cacheObjectContext()));
+
+ return sensitiveValMap.computeIfAbsent(wrapped, w ->
+ includeSensitive ?
+ ctx.unwrapBinaryIfNeeded(val, !deserializeBinary,
false, null) :
+ "[HIDDEN_VALUE#" + UUID.randomUUID() + "]");
Review comment:
UUID is just one of the possible choices here.
As I told you above, this seems to be the best choice.
The main goal is to have different values different even after masking, but
the same should still be the same.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
