[jira] [Commented] (QPID-7546) [Java Broker] Allow the system tests to be run using the Qpid JMS client for AMQP 1.0 testing

2017-01-15 Thread ASF subversion and git services (JIRA)

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

ASF subversion and git services commented on QPID-7546:
---

Commit 1778970 from [~k-wall] in branch 'java/trunk'
[ https://svn.apache.org/r1778970 ]

QPID-7546: Enable 
PublishMessageRestTest#testPublishMessageWithPropertiesAndHeaders for AMQP 1.0

> [Java Broker] Allow the system tests to be run using the Qpid JMS client for 
> AMQP 1.0 testing
> -
>
> Key: QPID-7546
> URL: https://issues.apache.org/jira/browse/QPID-7546
> Project: Qpid
>  Issue Type: Test
>  Components: Java Tests
>Reporter: Rob Godfrey
>Assignee: Rob Godfrey
> Attachments: 
> 0001-QPID-7546-Allow-overriding-of-jms-spec-artifact-in-o.patch
>
>
> Currently the system tests use the JMS client for AMQP 0-8/9/10 and so the 
> AMQP 1.0 protocol implementation does not get as thoroughly tested.
> We should aim to move all the system tests such that they can be run with 
> either client



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-7623) "SEVERE: RuntimeException while executing runnable" reported by Guava to stderr if a state transition method throws exception

2017-01-15 Thread Keith Wall (JIRA)

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

Keith Wall updated QPID-7623:
-
Description: 
As reported in QPID-7311, a state change that throws an exception, may cause a 
warning like the following to be reported to stderr:

{noformat}
SEVERE: RuntimeException while executing runnable 
com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
c5a6012
org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
get certificate for 'test' from 'https://google.com'
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.net.ConnectException: Connection timed out: connect
at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
at 
java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
at 
java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
at 
java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
at 
java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:579)
at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
... 5 more
{noformat}

The problem is a general one rather than being specific to SSTS.

The create exception handler involved code at lines 
AbstractConfiguredObject.java:1083 unconditionally re-raises the exception.  
The exception then goes uncaught and is logged by Guava at line 
ExecutionList.java:160.  Guava's behaviour here is documented in 
ListenableFuture.java:103.

Whilst ugly, this actually causes no functional harm.   The child creation 
error is still reported to the end user, and the child is still deregistered 
from the parent.  However, the appearance of the stack on stderr will cause 
alarm to end-users.

The cause of the problem is AbstractConfiguredObject.java:1083.  The 
{{FutureCallback#onFailure}} impl should not be rethrowing the exception.  
Setting returnValue's exception is sufficient.



  was:
As reported in QPID-7311, a state change that throws an exception, may cause a 
warning like the following to be reported to stderr:

{noformat}
SEVERE: RuntimeException while executing runnable 
com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
c5a6012
org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
get certificate for 'test' from 'https://google.com'
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.net.ConnectException: Connection timed out: connect
at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
at 
java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
at 
java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
at 
java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
at 
java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:579)
at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
... 5 more
{noformat}

The problem is a general one rather than being specific to SSTS.

The create exception handler involved code at lines 
AbstractConfiguredObject

[jira] [Updated] (QPID-7623) "SEVERE: RuntimeException while executing runnable" reported by Guava to stderr if a state transition method throws exception

2017-01-15 Thread Keith Wall (JIRA)

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

Keith Wall updated QPID-7623:
-
Status: Reviewable  (was: In Progress)

> "SEVERE: RuntimeException while executing runnable" reported by Guava to 
> stderr if a state transition method throws exception
> -
>
> Key: QPID-7623
> URL: https://issues.apache.org/jira/browse/QPID-7623
> Project: Qpid
>  Issue Type: Bug
>  Components: Java Broker
>Affects Versions: qpid-java-6.0, qpid-java-6.1
>Reporter: Keith Wall
>Assignee: Keith Wall
>Priority: Minor
> Fix For: qpid-java-7.0
>
>
> As reported in QPID-7311, a state change that throws an exception, may cause 
> a warning like the following to be reported to stderr:
> {noformat}
> SEVERE: RuntimeException while executing runnable 
> com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
> org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
> c5a6012
> org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
> get certificate for 'test' from 'https://google.com'
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
> at java.util.concurrent.FutureTask.run(FutureTask.java:262)
> at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> at java.lang.Thread.run(Thread.java:745)
> Caused by: java.net.ConnectException: Connection timed out: connect
> at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
> at 
> java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
> at 
> java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
> at 
> java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
> at 
> java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
> at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
> at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
> at java.net.Socket.connect(Socket.java:579)
> at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
> ... 5 more
> {noformat}
> The problem is a general one rather than being specific to SSTS.
> The create exception handler involved code at lines 
> AbstractConfiguredObject.java:1083 unconditional re-raises the exception.  
> The exception then goes uncaught and is logged by Guava at line 
> ExecutionList.java:160.  Guava's behaviour here is documented in 
> ListenableFuture.java:103.
> Whilst ugly, this actually causes no functional harm.   The child creation 
> error is still reported to the end user, and the child is still deregistered 
> from the parent.  However, the appearance of the stack on stderr will cause 
> alarm to end-users.
> The cause of the problem is AbstractConfiguredObject.java:1083.  The 
> {{FutureCallback#onFailure}} impl should not be rethrowing the exception.  
> Setting returnValue's exception is sufficient.



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-7623) "SEVERE: RuntimeException while executing runnable" reported by Guava to stderr if a state transition method throws exception

2017-01-15 Thread ASF subversion and git services (JIRA)

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

ASF subversion and git services commented on QPID-7623:
---

Commit 1778898 from [~k-wall] in branch 'java/trunk'
[ https://svn.apache.org/r1778898 ]

QPID-7623: [Java Broker] Prevent "SEVERE: RuntimeException while executing 
runnable" being reported by Guava if aactivation  state transition throws an 
exception during create child

> "SEVERE: RuntimeException while executing runnable" reported by Guava to 
> stderr if a state transition method throws exception
> -
>
> Key: QPID-7623
> URL: https://issues.apache.org/jira/browse/QPID-7623
> Project: Qpid
>  Issue Type: Bug
>  Components: Java Broker
>Affects Versions: qpid-java-6.0, qpid-java-6.1
>Reporter: Keith Wall
>Priority: Minor
> Fix For: qpid-java-7.0
>
>
> As reported in QPID-7311, a state change that throws an exception, may cause 
> a warning like the following to be reported to stderr:
> {noformat}
> SEVERE: RuntimeException while executing runnable 
> com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
> org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
> c5a6012
> org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
> get certificate for 'test' from 'https://google.com'
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
> at java.util.concurrent.FutureTask.run(FutureTask.java:262)
> at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> at java.lang.Thread.run(Thread.java:745)
> Caused by: java.net.ConnectException: Connection timed out: connect
> at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
> at 
> java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
> at 
> java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
> at 
> java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
> at 
> java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
> at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
> at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
> at java.net.Socket.connect(Socket.java:579)
> at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
> ... 5 more
> {noformat}
> The problem is a general one rather than being specific to SSTS.
> The create exception handler involved code at lines 
> AbstractConfiguredObject.java:1083 unconditional re-raises the exception.  
> The exception then goes uncaught and is logged by Guava at line 
> ExecutionList.java:160.  Guava's behaviour here is documented in 
> ListenableFuture.java:103.
> Whilst ugly, this actually causes no functional harm.   The child creation 
> error is still reported to the end user, and the child is still deregistered 
> from the parent.  However, the appearance of the stack on stderr will cause 
> alarm to end-users.
> The cause of the problem is AbstractConfiguredObject.java:1083.  The 
> {{FutureCallback#onFailure}} impl should not be rethrowing the exception.  
> Setting returnValue's exception is sufficient.



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Assigned] (QPID-7623) "SEVERE: RuntimeException while executing runnable" reported by Guava to stderr if a state transition method throws exception

2017-01-15 Thread Keith Wall (JIRA)

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

Keith Wall reassigned QPID-7623:


Assignee: Keith Wall

> "SEVERE: RuntimeException while executing runnable" reported by Guava to 
> stderr if a state transition method throws exception
> -
>
> Key: QPID-7623
> URL: https://issues.apache.org/jira/browse/QPID-7623
> Project: Qpid
>  Issue Type: Bug
>  Components: Java Broker
>Affects Versions: qpid-java-6.0, qpid-java-6.1
>Reporter: Keith Wall
>Assignee: Keith Wall
>Priority: Minor
> Fix For: qpid-java-7.0
>
>
> As reported in QPID-7311, a state change that throws an exception, may cause 
> a warning like the following to be reported to stderr:
> {noformat}
> SEVERE: RuntimeException while executing runnable 
> com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
> org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
> c5a6012
> org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
> get certificate for 'test' from 'https://google.com'
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
> at java.util.concurrent.FutureTask.run(FutureTask.java:262)
> at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> at java.lang.Thread.run(Thread.java:745)
> Caused by: java.net.ConnectException: Connection timed out: connect
> at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
> at 
> java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
> at 
> java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
> at 
> java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
> at 
> java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
> at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
> at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
> at java.net.Socket.connect(Socket.java:579)
> at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
> ... 5 more
> {noformat}
> The problem is a general one rather than being specific to SSTS.
> The create exception handler involved code at lines 
> AbstractConfiguredObject.java:1083 unconditional re-raises the exception.  
> The exception then goes uncaught and is logged by Guava at line 
> ExecutionList.java:160.  Guava's behaviour here is documented in 
> ListenableFuture.java:103.
> Whilst ugly, this actually causes no functional harm.   The child creation 
> error is still reported to the end user, and the child is still deregistered 
> from the parent.  However, the appearance of the stack on stderr will cause 
> alarm to end-users.
> The cause of the problem is AbstractConfiguredObject.java:1083.  The 
> {{FutureCallback#onFailure}} impl should not be rethrowing the exception.  
> Setting returnValue's exception is sufficient.



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-7623) "SEVERE: RuntimeException while executing runnable" reported by Guava to stderr if a state transition method throws exception

2017-01-15 Thread Keith Wall (JIRA)

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

Keith Wall updated QPID-7623:
-
Assignee: Alex Rudyy  (was: Keith Wall)

> "SEVERE: RuntimeException while executing runnable" reported by Guava to 
> stderr if a state transition method throws exception
> -
>
> Key: QPID-7623
> URL: https://issues.apache.org/jira/browse/QPID-7623
> Project: Qpid
>  Issue Type: Bug
>  Components: Java Broker
>Affects Versions: qpid-java-6.0, qpid-java-6.1
>Reporter: Keith Wall
>Assignee: Alex Rudyy
>Priority: Minor
> Fix For: qpid-java-7.0
>
>
> As reported in QPID-7311, a state change that throws an exception, may cause 
> a warning like the following to be reported to stderr:
> {noformat}
> SEVERE: RuntimeException while executing runnable 
> com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
> org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
> c5a6012
> org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
> get certificate for 'test' from 'https://google.com'
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
> at java.util.concurrent.FutureTask.run(FutureTask.java:262)
> at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> at java.lang.Thread.run(Thread.java:745)
> Caused by: java.net.ConnectException: Connection timed out: connect
> at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
> at 
> java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
> at 
> java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
> at 
> java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
> at 
> java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
> at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
> at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
> at java.net.Socket.connect(Socket.java:579)
> at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
> at 
> org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
> ... 5 more
> {noformat}
> The problem is a general one rather than being specific to SSTS.
> The create exception handler involved code at lines 
> AbstractConfiguredObject.java:1083 unconditional re-raises the exception.  
> The exception then goes uncaught and is logged by Guava at line 
> ExecutionList.java:160.  Guava's behaviour here is documented in 
> ListenableFuture.java:103.
> Whilst ugly, this actually causes no functional harm.   The child creation 
> error is still reported to the end user, and the child is still deregistered 
> from the parent.  However, the appearance of the stack on stderr will cause 
> alarm to end-users.
> The cause of the problem is AbstractConfiguredObject.java:1083.  The 
> {{FutureCallback#onFailure}} impl should not be rethrowing the exception.  
> Setting returnValue's exception is sufficient.



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-7311) [Java Broker] SiteSpecificTrustStore does not provide helpful error messages when protocol is missing

2017-01-15 Thread Keith Wall (JIRA)

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

Keith Wall commented on QPID-7311:
--

QPID-7623 captures the general problem you noticed.

On your other comments:

* The certificate downloading in a separate thread benefits the #doActivate 
when a _x509Certificate is not already stored.   Previously, the configuration 
thread would be blocked whilst the certificate was being download.  Now other 
sibling objects can acquire their desired state whilst this on downloads.
* I did consider changing the semantics of #refreshCertificate so that is 
merely requests the refresh, but in the end decided to stick with existing 
behaviour.
* I think in the long term long running operations such as this one or, a 
better example, Queue#clearQueue, should return a CancellableFutures.  These 
CancellableFutures should be exposed somehow to management so that operators 
can see the long running operations that are in-flight and choose to cancel 
them if they wish.   

> [Java Broker] SiteSpecificTrustStore does not provide helpful error messages 
> when protocol is missing
> -
>
> Key: QPID-7311
> URL: https://issues.apache.org/jira/browse/QPID-7311
> Project: Qpid
>  Issue Type: Improvement
>  Components: Java Broker
>Reporter: Lorenz Quack
>Assignee: Keith Wall
>Priority: Minor
> Fix For: qpid-java-7.0
>
>
> Currently, when creating a {{SiteSpecificTrustStore}} and the protocol is not 
> specified in the URL the object will be created but put into ERRORED state. 
> No indication as to what is wrong is given. Furthermore the URL is not 
> editable rendering the newly created object utterly useless.
> On the Broker side a MalformedURLException is logged at INFO level giving the 
> helpful information "no protocol".
> On the broker side we do not need a full stack trace.
> On the client side a error message would be preferable to creating an ERRORED 
> object.



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-7311) [Java Broker] SiteSpecificTrustStore does not provide helpful error messages when protocol is missing

2017-01-15 Thread Keith Wall (JIRA)

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

Keith Wall updated QPID-7311:
-
Status: Reviewable  (was: In Progress)

> [Java Broker] SiteSpecificTrustStore does not provide helpful error messages 
> when protocol is missing
> -
>
> Key: QPID-7311
> URL: https://issues.apache.org/jira/browse/QPID-7311
> Project: Qpid
>  Issue Type: Improvement
>  Components: Java Broker
>Reporter: Lorenz Quack
>Assignee: Keith Wall
>Priority: Minor
> Fix For: qpid-java-7.0
>
>
> Currently, when creating a {{SiteSpecificTrustStore}} and the protocol is not 
> specified in the URL the object will be created but put into ERRORED state. 
> No indication as to what is wrong is given. Furthermore the URL is not 
> editable rendering the newly created object utterly useless.
> On the Broker side a MalformedURLException is logged at INFO level giving the 
> helpful information "no protocol".
> On the broker side we do not need a full stack trace.
> On the client side a error message would be preferable to creating an ERRORED 
> object.



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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (QPID-7623) "SEVERE: RuntimeException while executing runnable" reported by Guava to stderr if a state transition method throws exception

2017-01-15 Thread Keith Wall (JIRA)
Keith Wall created QPID-7623:


 Summary: "SEVERE: RuntimeException while executing runnable" 
reported by Guava to stderr if a state transition method throws exception
 Key: QPID-7623
 URL: https://issues.apache.org/jira/browse/QPID-7623
 Project: Qpid
  Issue Type: Bug
  Components: Java Broker
Affects Versions: qpid-java-6.1, qpid-java-6.0
Reporter: Keith Wall
Priority: Minor
 Fix For: qpid-java-7.0


As reported in QPID-7311, a state change that throws an exception, may cause a 
warning like the following to be reported to stderr:

{noformat}
SEVERE: RuntimeException while executing runnable 
com.google.common.util.concurrent.Futures$6@5b83d60e with executor 
org.apache.qpid.server.configuration.updater.TaskExecutorImpl@2
c5a6012
org.apache.qpid.server.configuration.IllegalConfigurationException: Unable to 
get certificate for 'test' from 'https://google.com'
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:339)
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:302)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.net.ConnectException: Connection timed out: connect
at java.net.DualStackPlainSocketImpl.waitForConnect(Native Method)
at 
java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:85)
at 
java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
at 
java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
at 
java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:579)
at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:625)
at 
org.apache.qpid.server.security.SiteSpecificTrustStoreImpl$2.call(SiteSpecificTrustStoreImpl.java:316)
... 5 more
{noformat}

The problem is a general one rather than being specific to SSTS.

The create exception handler involved code at lines 
AbstractConfiguredObject.java:1083 unconditional re-raises the exception.  The 
exception then goes uncaught and is logged by Guava at line 
ExecutionList.java:160.  Guava's behaviour here is documented in 
ListenableFuture.java:103.

Whilst ugly, this actually causes no functional harm.   The child creation 
error is still reported to the end user, and the child is still deregistered 
from the parent.  However, the appearance of the stack on stderr will cause 
alarm to end-users.

The cause of the problem is AbstractConfiguredObject.java:1083.  The 
{{FutureCallback#onFailure}} impl should not be rethrowing the exception.  
Setting returnValue's exception is sufficient.





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

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org