karaf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (KARAF-6270) client could be more verbose on config error
Date Sun, 19 May 2019 04:57:00 GMT

    [ https://issues.apache.org/jira/browse/KARAF-6270?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16843311#comment-16843311

ASF subversion and git services commented on KARAF-6270:

Commit 3fca012f5eafe87792612d24380cef22a9760358 in karaf's branch refs/heads/master from Jean-Baptiste
[ https://gitbox.apache.org/repos/asf?p=karaf.git;h=3fca012 ]

Merge pull request #844 from jbonofre/KARAF-6270

[KARAF-6270] Using default value in ClientConfig when using invalid value

> client could be more verbose on config error
> --------------------------------------------
>                 Key: KARAF-6270
>                 URL: https://issues.apache.org/jira/browse/KARAF-6270
>             Project: Karaf
>          Issue Type: Bug
>    Affects Versions: 4.2.5
>            Reporter: Bernd Eckenfels
>            Assignee: Jean-Baptiste Onofré
>            Priority: Minor
>             Fix For: 4.3.0, 4.2.6
> When I have a "sshPort=" (empty) property then client.bat/sh will not use the default
port. Whats worse is, that the resulting error (even when -p is specified) does not tell me
what config file and what option is the problem:
>     Exception in thread "main" java.lang.NumberFormatException: For input string:
>             at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
>             at java.lang.Integer.parseInt(Integer.java:592)
>             at java.lang.Integer.parseInt(Integer.java:615)
>             at org.apache.karaf.client.ClientConfig.getInt(ClientConfig.java:238)
>             at org.apache.karaf.client.ClientConfig.<init>(ClientConfig.java:60)
>             at org.apache.karaf.client.Main.main(Main.java:76)
> I am willing to provide a patch which adds the property name to this exception. However
I think for trimmed empty properties it should fall back to the default as well.
> Question: what about invalid (non-numeric) content? Should it also fall back to the default
(asuming it might be even overwritten on the command line) as well or should we keep the (enhanced)

This message was sent by Atlassian JIRA

View raw message