isapego commented on a change in pull request #9610:
URL: https://github.com/apache/ignite/pull/9610#discussion_r757583818
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/odbc/ClientMessage.java
##########
@@ -35,6 +35,18 @@
/** */
private static final long serialVersionUID = -4609408156037304495L;
+ /** */
+ private static final int MAX_HANDSHAKE_SIZE = 1024 * 1024;
Review comment:
I'm not sure I like this solution as when we already implemented similia
solution in the past and very soon got a complaint from a user that had really
big password. Also, since that time we have introduced user attributes, which
also can be pretty large and I can easily imagine handshake exceeding 1 MB. If
we want to limit handshake size at least lets make it really big.
--
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]
