[ 
https://issues.apache.org/jira/browse/STORM-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15159784#comment-15159784
 ] 

ASF GitHub Bot commented on STORM-1254:
---------------------------------------

Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1139#discussion_r53862294
  
    --- Diff: storm-core/src/jvm/org/apache/storm/ui/UIHelpers.java ---
    @@ -0,0 +1,319 @@
    +/**
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + * <p/>
    + * http://www.apache.org/licenses/LICENSE-2.0
    + * <p/>
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.storm.ui;
    +
    +import clojure.lang.Keyword;
    +import clojure.lang.RT;
    +import com.google.common.base.Joiner;
    +import com.google.common.collect.ImmutableMap;
    +import com.google.common.collect.Lists;
    +import org.apache.commons.lang.StringEscapeUtils;
    +import org.apache.storm.generated.ExecutorInfo;
    +import org.apache.storm.logging.filters.AccessLoggingFilter;
    +import org.apache.storm.utils.Utils;
    +import org.eclipse.jetty.server.Connector;
    +import org.eclipse.jetty.server.DispatcherType;
    +import org.eclipse.jetty.server.Server;
    +import org.eclipse.jetty.server.nio.SelectChannelConnector;
    +import org.eclipse.jetty.server.ssl.SslSocketConnector;
    +import org.eclipse.jetty.servlet.FilterHolder;
    +import org.eclipse.jetty.servlet.FilterMapping;
    +import org.eclipse.jetty.servlet.ServletContextHandler;
    +import org.eclipse.jetty.servlet.ServletHolder;
    +import org.eclipse.jetty.servlets.CrossOriginFilter;
    +import org.eclipse.jetty.util.ssl.SslContextFactory;
    +import org.json.simple.JSONValue;
    +
    +import javax.servlet.Servlet;
    +import java.io.PrintWriter;
    +import java.io.StringWriter;
    +import java.net.URLEncoder;
    +import java.util.*;
    +
    +public class UIHelpers {
    +
    +    private static final String[][] PRETTY_SEC_DIVIDERS = {
    +            new String[]{"s", "60"},
    +            new String[]{"m", "60"},
    +            new String[]{"h", "24"},
    +            new String[]{"d", null}};
    +
    +    private static final String[][] PRETTY_MS_DIVIDERS = {
    +            new String[]{"ms", "1000"},
    +            new String[]{"s", "60"},
    +            new String[]{"m", "60"},
    +            new String[]{"h", "24"},
    +            new String[]{"d", null}};
    +
    +    public static String prettyUptimeStr(String val, String[][] dividers) {
    +        int uptime = Integer.parseInt(val);
    +        LinkedList<String> tmp = new LinkedList<>();
    +        for (String[] divider : dividers) {
    +            if (uptime > 0) {
    +                if (divider[1] != null) {
    +                    int div = Integer.parseInt(divider[1]);
    +                    tmp.addFirst(uptime % div + divider[0]);
    +                    uptime = uptime / div;
    +                } else {
    +                    tmp.addFirst(uptime + divider[0]);
    +                }
    +            }
    +        }
    +        return Joiner.on(" ").join(tmp);
    +    }
    +
    +    public static String prettyUptimeSec(String sec) {
    +        return prettyUptimeStr(sec, PRETTY_SEC_DIVIDERS);
    +    }
    +
    +    public static String prettyUptimeSec(int secs) {
    +        return prettyUptimeStr(String.valueOf(secs), PRETTY_SEC_DIVIDERS);
    +    }
    +
    +    public static String prettyUptimeMs(String ms) {
    +        return prettyUptimeStr(ms, PRETTY_MS_DIVIDERS);
    +    }
    +
    +    public static String prettyUptimeMs(int ms) {
    +        return prettyUptimeStr(String.valueOf(ms), PRETTY_MS_DIVIDERS);
    +    }
    +
    +
    +    public static String urlFormat(String fmt, Object... args) {
    +        String[] argsEncoded = new String[args.length];
    +        for (int i = 0; i < args.length; i++) {
    +            argsEncoded[i] = URLEncoder.encode(String.valueOf(args[i]));
    +        }
    +        return String.format(fmt, argsEncoded);
    +    }
    +
    +    public static String prettyExecutorInfo(ExecutorInfo e) {
    +        return "[" + e.get_task_start() + "-" + e.get_task_end() + "]";
    +    }
    +
    +    public static Map unauthorizedUserJson(String user) {
    +        return ImmutableMap.of(
    +                "error", "No Authorization",
    +                "errorMessage", String.format("User %s is not 
authorized.", user)
    +        );
    +    }
    +
    +    public static List unauthorizedUserHtml(String user) {
    +        return Lists.newArrayList(
    +                keyword("h1"),
    +                "User '",
    +                StringEscapeUtils.escapeHtml(user),
    +                "' is not authorized.");
    +    }
    +
    +    private static SslSocketConnector mkSslConnector(Integer port, String 
ksPath, String ksPassword, String ksType,
    +                                                     String keyPassword, 
String tsPath, String tsPassword, String tsType,
    +                                                     Boolean 
needClientAuth, Boolean wantClientAuth) {
    +        SslContextFactory factory = new SslContextFactory();
    +        factory.setExcludeCipherSuites("SSL_RSA_WITH_RC4_128_MD5", 
"SSL_RSA_WITH_RC4_128_SHA");
    +        factory.setExcludeProtocols("SSLv3");
    +        factory.setAllowRenegotiate(false);
    +        factory.setKeyStorePath(ksPath);
    +        factory.setKeyStoreType(ksType);
    +        factory.setKeyStorePassword(ksPassword);
    +        factory.setKeyManagerPassword(keyPassword);
    +
    +        if (tsPath != null && tsPassword != null && tsType != null) {
    +            factory.setTrustStore(tsPath);
    +            factory.setTrustStoreType(tsType);
    +            factory.setTrustStorePassword(tsPassword);
    +        }
    +
    +        if (needClientAuth != null && needClientAuth) {
    +            factory.setNeedClientAuth(true);
    +        }
    --- End diff --
    
    This should be an `else if`.  cond stops as soon as a condition is true.


> port backtype.storm.ui.helpers to java
> --------------------------------------
>
>                 Key: STORM-1254
>                 URL: https://issues.apache.org/jira/browse/STORM-1254
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Jark Wu
>              Labels: java-migration, jstorm-merger
>
> Helper functions for the UI/HTTP



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to