anshumg commented on a change in pull request #575:
URL: https://github.com/apache/solr/pull/575#discussion_r794758234
##########
File path:
solr/core/src/java/org/apache/solr/security/PKIAuthenticationPlugin.java
##########
@@ -75,6 +76,15 @@ public static void withServerIdentity(final boolean enabled)
{
}
private static final Logger log =
LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
+
+ /**
+ * If a number has less than this number of digits, it'll not be considered
a timestamp.
+ */
+ private static final int MIN_TIMESTAMP_DIGITS = 10; // a timestamp of
9999999999 is year 1970
Review comment:
This range looks reasonable to parse a timestamp from now and years to
come.
Considering it's expected to be a timestamp, we could have this be between
13 and 14 digits but guess that's just optimization that'd not really help with
anything. Let's leave it at this :)
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
