Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2430#discussion_r153232875 --- Diff: storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java --- @@ -97,8 +98,14 @@ public LogviewerLogSearchHandler(Map<String, Object> stormConf, String logRoot, this.logRoot = logRoot; this.daemonLogRoot = daemonLogRoot; this.resourceAuthorizer = resourceAuthorizer; - - this.logviewerPort = ObjectReader.getInt(stormConf.get(DaemonConfig.LOGVIEWER_PORT)); + Integer httpsPort = ObjectReader.getInt(stormConf.get(DaemonConfig.LOGVIEWER_HTTPS_PORT), 0); --- End diff -- @Ethanlm I don't understand why? As far as I can tell both `@isInteger` (https://github.com/apache/storm/blob/master/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java#L160) and `@isPositiveNumber` (https://github.com/apache/storm/blob/master/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java#L450) allow nulls to pass the validation check. The configuration we'd disallow would be setting the https port to a negative number, which seems reasonable enough to me.
---