[ 
https://issues.apache.org/jira/browse/STORM-2827?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

ASF GitHub Bot updated STORM-2827:
----------------------------------
    Labels: pull-request-available  (was: )

> Logviewer search returns incorrect logviewerUrl
> -----------------------------------------------
>
>                 Key: STORM-2827
>                 URL: https://issues.apache.org/jira/browse/STORM-2827
>             Project: Apache Storm
>          Issue Type: Bug
>            Reporter: Ethan Li
>            Assignee: Ethan Li
>            Priority: Minor
>              Labels: pull-request-available
>
> Code in LogviewerLogSearchHandler
> {code:java}
>   @VisibleForTesting
>     String urlToMatchCenteredInLogPage(byte[] needle, Path canonicalPath, int 
> offset, Integer port) throws UnknownHostException {
>         final String host = Utils.hostname();
>         final Path truncatedFilePath = 
> truncatePathToLastElements(canonicalPath, 3);
>         Map<String, Object> parameters = new HashMap<>();
>         parameters.put("file", truncatedFilePath.toString());
>         parameters.put("start", Math.max(0, offset - 
> (LogviewerConstant.DEFAULT_BYTES_PER_PAGE / 2) - (needle.length / -2)));
>         parameters.put("length", LogviewerConstant.DEFAULT_BYTES_PER_PAGE);
>         return UrlBuilder.build(String.format("http://%s:%d/api/v1/log";, 
> host, port), parameters);
>     }
>     @VisibleForTesting
>     String urlToMatchCenteredInLogPageDaemonFile(byte[] needle, Path 
> canonicalPath, int offset, Integer port) throws UnknownHostException {
>         final String host = Utils.hostname();
>         final Path truncatedFilePath = 
> truncatePathToLastElements(canonicalPath, 1);
>         Map<String, Object> parameters = new HashMap<>();
>         parameters.put("file", truncatedFilePath.toString());
>         parameters.put("start", Math.max(0, offset - 
> (LogviewerConstant.DEFAULT_BYTES_PER_PAGE / 2) - (needle.length / -2)));
>         parameters.put("length", LogviewerConstant.DEFAULT_BYTES_PER_PAGE);
>         return 
> UrlBuilder.build(String.format("http://%s:%d/api/v1/daemonlog";, host, port), 
> parameters);
>     }
> {code}
> only returns http url. This url will be invalid if logviewer https port is 
> configured, in which case the http url will be not found



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to