[jira] [Commented] (DISPATCH-1160) Add edge address tracking module to interior routers which will inform edges of mobile address receiver changes

2018-10-30 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1160:
--

GitHub user ganeshmurthy opened a pull request:

https://github.com/apache/qpid-dispatch/pull/410

DISPATCH-1160 - Added edge address tracking module to interior router



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ganeshmurthy/qpid-dispatch EDGE-ADDR-TRACKING

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/410.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #410


commit cf5f9b7f958fa88ca747b4ec29a1cf2b0932aa33
Author: Ganesh Murthy 
Date:   2018-10-18T15:46:42Z

DISPATCH-1160 - Added edge address tracking module to interior router




> Add edge address tracking module to interior routers which will inform edges 
> of mobile address receiver changes
> ---
>
> Key: DISPATCH-1160
> URL: https://issues.apache.org/jira/browse/DISPATCH-1160
> Project: Qpid Dispatch
>  Issue Type: Improvement
>Affects Versions: 1.4.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.5.0
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Resolved] (PROTON-1959) [c,cpp] API additions to simplify reconnect

2018-10-30 Thread Alan Conway (JIRA)


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

Alan Conway resolved PROTON-1959.
-
Resolution: Fixed

> [c,cpp] API additions to simplify reconnect
> ---
>
> Key: PROTON-1959
> URL: https://issues.apache.org/jira/browse/PROTON-1959
> Project: Qpid Proton
>  Issue Type: Improvement
>Affects Versions: proton-c-0.25.0
>Reporter: Alan Conway
>Assignee: Alan Conway
>Priority: Major
> Fix For: proton-c-0.26.0
>
>
> Add some minor API extensions to make it easier to tell if a connection is 
> reconnecting, and to set up one-time events that only happen on the initial 
> connect.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[GitHub] qpid-dispatch pull request #410: DISPATCH-1160 - Added edge address tracking...

2018-10-30 Thread ganeshmurthy
GitHub user ganeshmurthy opened a pull request:

https://github.com/apache/qpid-dispatch/pull/410

DISPATCH-1160 - Added edge address tracking module to interior router



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ganeshmurthy/qpid-dispatch EDGE-ADDR-TRACKING

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/410.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #410


commit cf5f9b7f958fa88ca747b4ec29a1cf2b0932aa33
Author: Ganesh Murthy 
Date:   2018-10-18T15:46:42Z

DISPATCH-1160 - Added edge address tracking module to interior router




---

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



[jira] [Commented] (PROTON-1959) [c,cpp] API additions to simplify reconnect

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on PROTON-1959:
-

Commit 77ddf4b03afb89a21e29698fcd90117e4d63b314 in qpid-proton's branch 
refs/heads/master from [~aconway]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=77ddf4b ]

PROTON-1959: [cpp] Zero pn_listener_t* in  proton::listener::stop()

Pointer becomes invalid sometime after stop() so must not be used again.


> [c,cpp] API additions to simplify reconnect
> ---
>
> Key: PROTON-1959
> URL: https://issues.apache.org/jira/browse/PROTON-1959
> Project: Qpid Proton
>  Issue Type: Improvement
>Affects Versions: proton-c-0.25.0
>Reporter: Alan Conway
>Assignee: Alan Conway
>Priority: Major
> Fix For: proton-c-0.26.0
>
>
> Add some minor API extensions to make it easier to tell if a connection is 
> reconnecting, and to set up one-time events that only happen on the initial 
> connect.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (PROTON-1959) [c,cpp] API additions to simplify reconnect

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on PROTON-1959:
-

Commit 6e9bbe99326ad3e674f8b9701b3bd07c46b744ef in qpid-proton's branch 
refs/heads/master from [~aconway]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=6e9bbe9 ]

PROTON-1959: [cpp] Update simple_send example to re-send on reconnect


> [c,cpp] API additions to simplify reconnect
> ---
>
> Key: PROTON-1959
> URL: https://issues.apache.org/jira/browse/PROTON-1959
> Project: Qpid Proton
>  Issue Type: Improvement
>Affects Versions: proton-c-0.25.0
>Reporter: Alan Conway
>Assignee: Alan Conway
>Priority: Major
> Fix For: proton-c-0.26.0
>
>
> Add some minor API extensions to make it easier to tell if a connection is 
> reconnecting, and to set up one-time events that only happen on the initial 
> connect.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (PROTON-1959) [c,cpp] API additions to simplify reconnect

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on PROTON-1959:
-

Commit 84b0129d636841d3b1e7f4cd76eb84b8d00163bf in qpid-proton's branch 
refs/heads/master from [~aconway]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=84b0129 ]

PROTON-1959: [c epoll] Fix crash in some reconnect scenarios.

Proactor should release driver->collector *not* pn_connection_collector()
as the driver always keeps a valid collector pointer, but the connection may be
NULL if it has been released for reconnect.


> [c,cpp] API additions to simplify reconnect
> ---
>
> Key: PROTON-1959
> URL: https://issues.apache.org/jira/browse/PROTON-1959
> Project: Qpid Proton
>  Issue Type: Improvement
>Affects Versions: proton-c-0.25.0
>Reporter: Alan Conway
>Assignee: Alan Conway
>Priority: Major
> Fix For: proton-c-0.26.0
>
>
> Add some minor API extensions to make it easier to tell if a connection is 
> reconnecting, and to set up one-time events that only happen on the initial 
> connect.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (PROTON-1959) [c,cpp] API additions to simplify reconnect

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on PROTON-1959:
-

Commit f53c7683d7e903634b53db30d1f401006b43015a in qpid-proton's branch 
refs/heads/master from [~aconway]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=f53c768 ]

PROTON-1959: [cpp] API additions to simplify reconnect

messaging_handler::on_connection_start() - initial, exactly once event
messaging_handler::on_connection_reconnecting() - disconnected with 
auto-reconnect pending
bool connection::reconnected() - connection has been auto-reconnected

Added connection life-cycle doc to the messaging_handler class API doc.


> [c,cpp] API additions to simplify reconnect
> ---
>
> Key: PROTON-1959
> URL: https://issues.apache.org/jira/browse/PROTON-1959
> Project: Qpid Proton
>  Issue Type: Improvement
>Affects Versions: proton-c-0.25.0
>Reporter: Alan Conway
>Assignee: Alan Conway
>Priority: Major
> Fix For: proton-c-0.26.0
>
>
> Add some minor API extensions to make it easier to tell if a connection is 
> reconnecting, and to set up one-time events that only happen on the initial 
> connect.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Created] (DISPATCH-1161) Handle edge routers in the console

2018-10-30 Thread Ernest Allen (JIRA)
Ernest Allen created DISPATCH-1161:
--

 Summary: Handle edge routers in the console
 Key: DISPATCH-1161
 URL: https://issues.apache.org/jira/browse/DISPATCH-1161
 Project: Qpid Dispatch
  Issue Type: Improvement
  Components: Console
Affects Versions: 1.4.1
Reporter: Ernest Allen
Assignee: Ernest Allen


Show edge routers on the console's topology page.
 * Show them on the diagram in a unique color or size
 * Add them to the legend
 * Consolidate mulitple edge routers attached to a router as a group icon
 * Whem mouseover link between router and edge routers, show traffic stats for 
busiest edge routers



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPIDJMS-420) Improve performance of MessageConsumer processing

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on QPIDJMS-420:
-

Commit 24d03437fb1695b2096989c3561007312d4169ee in qpid-jms's branch 
refs/heads/master from [~tabish121]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-jms.git;h=24d0343 ]

QPIDJMS-420 Use local tracking in AmqpConsumer for prefetch refill

Track the number of message that are currently in the prefetch buffer using
local counters that mesh well with the already added delivered message counts
to reduce access to synchronized code in the message queue.  Reduce code paths
where a message settlement might be missed or credit not replenished on error.


> Improve performance of MessageConsumer processing
> -
>
> Key: QPIDJMS-420
> URL: https://issues.apache.org/jira/browse/QPIDJMS-420
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Timothy Bish
>Assignee: Timothy Bish
>Priority: Minor
> Fix For: 0.38.0
>
>
> Refactor some of the code paths that handling inbound message processing and 
> eventually queue or deliver inbound messages to the MessageConsumer.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (DISPATCH-1160) Add edge address tracking module to interior routers which will inform edges of mobile address receiver changes

2018-10-30 Thread Ganesh Murthy (JIRA)


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

Ganesh Murthy updated DISPATCH-1160:

Environment: (was: Create a interior router module called edge address 
tracking. This module will allow a receiver link (Edge Address Tracking Link - 
EATL)  from interior to edge to be created on hard coded address 
_$qd.edge_addr_tracking

When receivers are added/removed to a mobile address, this module will inform 
the edge router if necessary via the EATL allowing the edge router to act 
accordingly.

This module will act only on mobile addresses.)

> Add edge address tracking module to interior routers which will inform edges 
> of mobile address receiver changes
> ---
>
> Key: DISPATCH-1160
> URL: https://issues.apache.org/jira/browse/DISPATCH-1160
> Project: Qpid Dispatch
>  Issue Type: Improvement
>Affects Versions: 1.4.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.5.0
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (DISPATCH-1160) Add edge address tracking module to interior routers which will inform edges of mobile address receiver changes

2018-10-30 Thread Ganesh Murthy (JIRA)


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

Ganesh Murthy commented on DISPATCH-1160:
-

Create a interior router module called edge address tracking. This module will 
allow a receiver link (Edge Address Tracking Link - EATL)  from interior to 
edge to be created on hard coded address _$qd.edge_addr_tracking

When receivers are added/removed to a mobile address, this module will inform 
the edge router if necessary via the EATL allowing the edge router to act 
accordingly.

This module will act only on mobile addresses.

> Add edge address tracking module to interior routers which will inform edges 
> of mobile address receiver changes
> ---
>
> Key: DISPATCH-1160
> URL: https://issues.apache.org/jira/browse/DISPATCH-1160
> Project: Qpid Dispatch
>  Issue Type: Improvement
>Affects Versions: 1.4.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.5.0
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Created] (DISPATCH-1160) Add edge address tracking module to interior routers which will inform edges of mobile address receiver changes

2018-10-30 Thread Ganesh Murthy (JIRA)
Ganesh Murthy created DISPATCH-1160:
---

 Summary: Add edge address tracking module to interior routers 
which will inform edges of mobile address receiver changes
 Key: DISPATCH-1160
 URL: https://issues.apache.org/jira/browse/DISPATCH-1160
 Project: Qpid Dispatch
  Issue Type: Improvement
Affects Versions: 1.4.1
 Environment: Create a interior router module called edge address 
tracking. This module will allow a receiver link (Edge Address Tracking Link - 
EATL)  from interior to edge to be created on hard coded address 
_$qd.edge_addr_tracking

When receivers are added/removed to a mobile address, this module will inform 
the edge router if necessary via the EATL allowing the edge router to act 
accordingly.

This module will act only on mobile addresses.
Reporter: Ganesh Murthy
Assignee: Ganesh Murthy
 Fix For: 1.5.0






--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (DISPATCH-1159) Remove the term "uplink" from the edge router - It's confusing

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on DISPATCH-1159:
---

Commit 413d727b6e1772d87a23f88261cdced4940d33ee in qpid-dispatch's branch 
refs/heads/master from [~tr...@redhat.com]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=413d727 ]

DISPATCH-1159 - Removed the term "uplink" from the edge router implementation.


> Remove the term "uplink" from the edge router - It's confusing
> --
>
> Key: DISPATCH-1159
> URL: https://issues.apache.org/jira/browse/DISPATCH-1159
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Router Node
>Reporter: Ted Ross
>Assignee: Ted Ross
>Priority: Major
> Fix For: 1.5.0
>
>
> Renamed "edge_uplink" to "edge_connection".
> Changed "edge_uplink" connection role to "edge".
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Resolved] (DISPATCH-1159) Remove the term "uplink" from the edge router - It's confusing

2018-10-30 Thread Ted Ross (JIRA)


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

Ted Ross resolved DISPATCH-1159.

Resolution: Fixed

> Remove the term "uplink" from the edge router - It's confusing
> --
>
> Key: DISPATCH-1159
> URL: https://issues.apache.org/jira/browse/DISPATCH-1159
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Router Node
>Reporter: Ted Ross
>Assignee: Ted Ross
>Priority: Major
> Fix For: 1.5.0
>
>
> Renamed "edge_uplink" to "edge_connection".
> Changed "edge_uplink" connection role to "edge".
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Created] (DISPATCH-1159) Remove the term "uplink" from the edge router - It's confusing

2018-10-30 Thread Ted Ross (JIRA)
Ted Ross created DISPATCH-1159:
--

 Summary: Remove the term "uplink" from the edge router - It's 
confusing
 Key: DISPATCH-1159
 URL: https://issues.apache.org/jira/browse/DISPATCH-1159
 Project: Qpid Dispatch
  Issue Type: Improvement
  Components: Router Node
Reporter: Ted Ross
Assignee: Ted Ross
 Fix For: 1.5.0


Renamed "edge_uplink" to "edge_connection".

Changed "edge_uplink" connection role to "edge".

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-8254) [Broker-J] Illegal ascii characters are used in keystore passwords

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit cc4821e344a0d23e71a4e4fcc8caf6cd75827c1a in qpid-broker-j's branch 
refs/heads/7.0.x from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=cc4821e ]

QPID-8254: [Broker-J] Stop using illegal ascii characters in java keystore 
passwords

(cherry picked from commit 16b5592f12ccb841e432f5e16c4851c21ccc66cf)


> [Broker-J] Illegal ascii characters are used in keystore passwords
> --
>
> Key: QPID-8254
> URL: https://issues.apache.org/jira/browse/QPID-8254
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.0.8, qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> Illegal keystore password characters are used on creation of 
> {{AutoGeneratedSelfSignedKeyStoreImpl}} and {{NonJavaKeyStoreImpl}}:
> {code}
> char[] chars = new char[64];
> RANDOM.nextBytes(bytes);
> StandardCharsets.US_ASCII.decode(ByteBuffer.wrap(bytes)).get(chars);
> inMemoryKeyStore.load(null, chars);
> inMemoryKeyStore.setKeyEntry("1", _privateKey, chars, certs);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-8254) [Broker-J] Illegal ascii characters are used in keystore passwords

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-8254:
-
Status: Reviewable  (was: In Progress)

> [Broker-J] Illegal ascii characters are used in keystore passwords
> --
>
> Key: QPID-8254
> URL: https://issues.apache.org/jira/browse/QPID-8254
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.0.8, qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> Illegal keystore password characters are used on creation of 
> {{AutoGeneratedSelfSignedKeyStoreImpl}} and {{NonJavaKeyStoreImpl}}:
> {code}
> char[] chars = new char[64];
> RANDOM.nextBytes(bytes);
> StandardCharsets.US_ASCII.decode(ByteBuffer.wrap(bytes)).get(chars);
> inMemoryKeyStore.load(null, chars);
> inMemoryKeyStore.setKeyEntry("1", _privateKey, chars, certs);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-8253) [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned from SSLEngine.getSSLParameters().getServerNames()

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-8253:
-
Status: Reviewable  (was: In Progress)

> [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned 
> from SSLEngine.getSSLParameters().getServerNames()
> --
>
> Key: QPID-8253
> URL: https://issues.apache.org/jira/browse/QPID-8253
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> QpidBestFitX509KeyManager does not handle null server names returned from 
> SSLENgine.getSSLParameters().getServerNames()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-8253) [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned from SSLEngine.getSSLParameters().getServerNames()

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-8253:
-
Fix Version/s: (was: qpid-java-broker-7.0.7)

> [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned 
> from SSLEngine.getSSLParameters().getServerNames()
> --
>
> Key: QPID-8253
> URL: https://issues.apache.org/jira/browse/QPID-8253
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> QpidBestFitX509KeyManager does not handle null server names returned from 
> SSLENgine.getSSLParameters().getServerNames()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Assigned] (QPID-8253) [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned from SSLEngine.getSSLParameters().getServerNames()

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy reassigned QPID-8253:


Assignee: Alex Rudyy

> [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned 
> from SSLEngine.getSSLParameters().getServerNames()
> --
>
> Key: QPID-8253
> URL: https://issues.apache.org/jira/browse/QPID-8253
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> QpidBestFitX509KeyManager does not handle null server names returned from 
> SSLENgine.getSSLParameters().getServerNames()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Assigned] (QPID-8254) [Broker-J] Illegal ascii characters are used in keystore passwords

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy reassigned QPID-8254:


Assignee: Alex Rudyy

> [Broker-J] Illegal ascii characters are used in keystore passwords
> --
>
> Key: QPID-8254
> URL: https://issues.apache.org/jira/browse/QPID-8254
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.0.8, qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> Illegal keystore password characters are used on creation of 
> {{AutoGeneratedSelfSignedKeyStoreImpl}} and {{NonJavaKeyStoreImpl}}:
> {code}
> char[] chars = new char[64];
> RANDOM.nextBytes(bytes);
> StandardCharsets.US_ASCII.decode(ByteBuffer.wrap(bytes)).get(chars);
> inMemoryKeyStore.load(null, chars);
> inMemoryKeyStore.setKeyEntry("1", _privateKey, chars, certs);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Resolved] (QPID-8132) Refresh Apache BCEL dependency to 6.2

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy resolved QPID-8132.
--
Resolution: Fixed

> Refresh Apache BCEL dependency to 6.2
> -
>
> Key: QPID-8132
> URL: https://issues.apache.org/jira/browse/QPID-8132
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Keith Wall
>Assignee: Alex Rudyy
>Priority: Minor
> Fix For: qpid-java-broker-7.1.0
>
>
> The Apache BCEL dependency used by Broker-J is getting stale (5.2).  6.2 is 
> the latest release.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Resolved] (QPID-8196) [Broker-J] [WMC] Edit form controls are not populated with current values

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy resolved QPID-8196.
--
Resolution: Fixed

> [Broker-J] [WMC] Edit form controls are not populated with current values
> -
>
> Key: QPID-8196
> URL: https://issues.apache.org/jira/browse/QPID-8196
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.1.0, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: 
> 0001-QPID-8196-Broker-J-WMC-Make-sure-that-edit-form-cont.patch
>
>
> The form for editing BDB HA VHN does not populate reliably the UI controls 
> with current attribute values. Some of the attribute controls can be left 
> unset. As result, on submitting the form, null/empty values can be sent as 
> part of update request to the broker.
> User impact:
> Editing of attributes cannot be done reliably from WVM, as when the issue 
> occurs, the update requests from WMC fail due to broker side validation. 
> There is a possibility of unexpected change of attribute value to null.
> Work around: REST API can be used to change the attribute values.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Resolved] (QPID-8216) [Broker-J] Operational log message CHN-1011 about moving message to dead letter queue is not reported

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy resolved QPID-8216.
--
Resolution: Fixed

> [Broker-J] Operational log message CHN-1011 about moving message to dead 
> letter queue is not reported
> -
>
> Key: QPID-8216
> URL: https://issues.apache.org/jira/browse/QPID-8216
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.0.3, qpid-java-broker-7.0.2, 
> qpid-java-broker-7.0.0, qpid-java-broker-7.0.1, qpid-java-broker-7.0.4, 
> qpid-java-broker-7.0.5, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> Operational log message CHN-1011 about moving message to dead letter queue is 
> not reported any more. The changes committed in [ 
> https://svn.apache.org/r1776037 ] against QPID-6028 removed reporting of 
> operational log message CHN-1011:
> {noformat}
>  CHN-1011 : Message :  moved to dead letter queue :  name>
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-7885:
-
Status: Reviewable  (was: In Progress)

> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> The task of modularization of  broker-j  is excluded from this JIRA, as it 
> seems, that many existing modules need to export the same packages which is 
> not permitted by Jigsaw module system. Some classes need to be moved into 
> separate packages which requires more substantial refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit 73745fe8be84dd3b94e618b19c44dccbdee0d71f in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=73745fe ]

QPID-7885: [Broker-J] Update PluggableProcessor to handle NoSuchFileException 
on generation of service files


> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> The task of modularization of  broker-j  is excluded from this JIRA, as it 
> seems, that many existing modules need to export the same packages which is 
> not permitted by Jigsaw module system. Some classes need to be moved into 
> separate packages which requires more substantial refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit a045ea8a749fa3ee6a76e6398b35d39b4bf8af76 in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=a045ea8 ]

QPID-7885: Fix failing tests


> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> The task of modularization of  broker-j  is excluded from this JIRA, as it 
> seems, that many existing modules need to export the same packages which is 
> not permitted by Jigsaw module system. Some classes need to be moved into 
> separate packages which requires more substantial refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit 35998c406fd22606c60202308ecad75cfed98f84 in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=35998c4 ]

QPID-7885: [Broker-J] Use jaxb dependency with legacy JMS client as it depends 
on jaxb base64 encoding


> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> The task of modularization of  broker-j  is excluded from this JIRA, as it 
> seems, that many existing modules need to export the same packages which is 
> not permitted by Jigsaw module system. Some classes need to be moved into 
> separate packages which requires more substantial refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit 3ff81dac366e77c99564e5434d5679fe9f835d02 in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=3ff81da ]

QPID-7885: [Broker-J] Upgrade apache parent pom to version 21 and update maven 
plugin versions


> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> The task of modularization of  broker-j  is excluded from this JIRA, as it 
> seems, that many existing modules need to export the same packages which is 
> not permitted by Jigsaw module system. Some classes need to be moved into 
> separate packages which requires more substantial refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-8253) [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned from SSLEngine.getSSLParameters().getServerNames()

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit 4ebae1747328e19519fa670974dfac92fb852a71 in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=4ebae17 ]

QPID-8253: [Broker-J] Hanlde null server names returned from SSLEngine 
SSLParameters


> [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned 
> from SSLEngine.getSSLParameters().getServerNames()
> --
>
> Key: QPID-8253
> URL: https://issues.apache.org/jira/browse/QPID-8253
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> QpidBestFitX509KeyManager does not handle null server names returned from 
> SSLENgine.getSSLParameters().getServerNames()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (QPID-8254) [Broker-J] Illegal ascii characters are used in keystore passwords

2018-10-30 Thread ASF subversion and git services (JIRA)


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

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

Commit 16b5592f12ccb841e432f5e16c4851c21ccc66cf in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=16b5592 ]

QPID-8254: [Broker-J] Stop using illegal ascii characters in java keystore 
passwords


> [Broker-J] Illegal ascii characters are used in keystore passwords
> --
>
> Key: QPID-8254
> URL: https://issues.apache.org/jira/browse/QPID-8254
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.0.8, qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> Illegal keystore password characters are used on creation of 
> {{AutoGeneratedSelfSignedKeyStoreImpl}} and {{NonJavaKeyStoreImpl}}:
> {code}
> char[] chars = new char[64];
> RANDOM.nextBytes(bytes);
> StandardCharsets.US_ASCII.decode(ByteBuffer.wrap(bytes)).get(chars);
> inMemoryKeyStore.load(null, chars);
> inMemoryKeyStore.setKeyEntry("1", _privateKey, chars, certs);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Created] (QPID-8255) [JMS AMQP 0-x] Connection failure with connection option 'client_cert_priv_key_path' and java 11

2018-10-30 Thread Alex Rudyy (JIRA)
Alex Rudyy created QPID-8255:


 Summary: [JMS AMQP 0-x] Connection failure  with connection option 
'client_cert_priv_key_path' and java 11
 Key: QPID-8255
 URL: https://issues.apache.org/jira/browse/QPID-8255
 Project: Qpid
  Issue Type: Bug
  Components: JMS AMQP 0-x
Affects Versions: qpid-java-client-0-x-6.3.3
 Environment: java11
Reporter: Alex Rudyy


Connection establishment fails with java 11 when connection option 
'client_cert_priv_key_path' is used:

{noformat}

javax.jms.JMSException: Error creating connection: Key protection  algorithm 
not found: java.security.UnrecoverableKeyException: Encrypt Private Key failed: 
getSecretKey failed: Password is not ASCII

    at 
org.apache.qpid.client.AMQConnectionFactory.createConnection(AMQConnectionFactory.java:169)
    at 
org.apache.qpid.client.AMQConnectionFactory.createConnection(AMQConnectionFactory.java:143)
    at 
org.apache.qpid.client.AMQConnectionFactory.createConnection(AMQConnectionFactory.java:57)
    at 
org.apache.qpid.systests.QpidJmsClient0xConnectionBuilder.build(QpidJmsClient0xConnectionBuilder.java:270)
    at 
org.apache.qpid.systests.jms_1_1.extensions.tls.TlsTest.testCreateSSLWithCertFileAndPrivateKey(TlsTest.java:508)
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native 
Method)
    at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at 
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.base/java.lang.reflect.Method.invoke(Method.java:566)
    at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
    at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
    at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
    at org.junit.rules.RunRules.evaluate(RunRules.java:20)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
    at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
    at 
org.apache.qpid.tests.utils.QpidTestRunner.runChild(QpidTestRunner.java:84)
    at 
org.apache.qpid.tests.utils.QpidTestRunner.runChild(QpidTestRunner.java:28)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
    at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
    at 
org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
    at org.apache.qpid.tests.utils.QpidTestRunner.run(QpidTestRunner.java:59)
    at org.junit.runner.JUnitCore.run(JUnitCore.java:160)
    at 
com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:68)
    at 
com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:47)
    at 
com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:242)
    at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70)
Caused by: org.apache.qpid.AMQConnectionFailureException: Key protection  
algorithm not found: java.security.UnrecoverableKeyException: Encrypt Private 
Key failed: getSecretKey failed: Password is not ASCII
    at 
org.apache.qpid.client.AMQConnection.makeConnection(AMQConnection.java:672)
    at org.apache.qpid.client.AMQConnection.(AMQConnection.java:522)
    at 
org.apache.qpid.client.AMQConnectionFactory.createConnection(AMQConnectionFactory.java:164)
    ... 32 more
Caused by: org.apache.qpid.transport.TransportException: Error creating SSL 
Context
    at 
org.apache.qpid.transport.network.security.SecurityLayerFactory$SSLSecurityLayer.(SecurityLayerFactory.java:100)
    at 
org.apache.qpid.transport.network.security.SecurityLayerFactory.newInstance(SecurityLayerFactory.java:59)
    at 
org.apache.qpid.client.AMQConnectionDelegate_8_0.makeBrokerConnection(AMQConnectionDelegate_8_0.java:142)
    at 
org.apache.qpid.client.AMQConnection.makeBrokerConnection(AMQConnection.java:816)
    at 
org.apache.qpid.client.AMQConnection.makeConnection(AMQConnection.java:583)
    ... 34 more
Caused by: java.security.KeyStoreException: Key protection  algorithm not 
found: java.security.UnrecoverableKeyException: Encrypt Private Key failed: 
getSecretKey failed: Password is not ASCII
    at 
java.base/sun.security.pkcs12.PKCS12KeyStore.setKe

[jira] [Created] (QPID-8254) [Broker-J] Illegal ascii characters are used in keystore passwords

2018-10-30 Thread Alex Rudyy (JIRA)
Alex Rudyy created QPID-8254:


 Summary: [Broker-J] Illegal ascii characters are used in keystore 
passwords
 Key: QPID-8254
 URL: https://issues.apache.org/jira/browse/QPID-8254
 Project: Qpid
  Issue Type: Bug
  Components: Broker-J
Affects Versions: qpid-java-broker-7.0.6, qpid-java-6.1.7, qpid-java-6.0.8
Reporter: Alex Rudyy
 Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7


Illegal keystore password characters are used on creation of 
{{AutoGeneratedSelfSignedKeyStoreImpl}} and {{NonJavaKeyStoreImpl}}:

{code}
char[] chars = new char[64];
RANDOM.nextBytes(bytes);
StandardCharsets.US_ASCII.decode(ByteBuffer.wrap(bytes)).get(chars);
inMemoryKeyStore.load(null, chars);
inMemoryKeyStore.setKeyEntry("1", _privateKey, chars, certs);
{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (PROTON-1938) misaligned Transport set/getErrorCondition and closed() behaviour

2018-10-30 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on PROTON-1938:


Github user gemmellr commented on the issue:

https://github.com/apache/qpid-proton-j/pull/16
  
As noted on the JIRA, an alternate change was made in 
19bf22f7e9b88e73db4b195f674d1527dbd713f3.


> misaligned Transport set/getErrorCondition and closed() behaviour
> -
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[GitHub] qpid-proton-j issue #16: PROTON-1938: Fix error propagation in TransportImpl

2018-10-30 Thread gemmellr
Github user gemmellr commented on the issue:

https://github.com/apache/qpid-proton-j/pull/16
  
As noted on the JIRA, an alternate change was made in 
19bf22f7e9b88e73db4b195f674d1527dbd713f3.


---

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



[jira] [Commented] (PROTON-1938) misaligned Transport set/getErrorCondition and closed() behaviour

2018-10-30 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on PROTON-1938:


Github user asfgit closed the pull request at:

https://github.com/apache/qpid-proton-j/pull/16


> misaligned Transport set/getErrorCondition and closed() behaviour
> -
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (PROTON-1938) misaligned Transport set/getErrorCondition and closed() behaviour

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on PROTON-1938:
-

Commit 2951b4051466a4c0ee888a5f8fa012c62ed138d3 in qpid-proton-j's branch 
refs/heads/master from [~gemmellr]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton-j.git;h=2951b40 ]

PROTON-1938: cleanup PR, alternate change made in 
19bf22f7e9b88e73db4b195f674d1527dbd713f3. This closes #16


> misaligned Transport set/getErrorCondition and closed() behaviour
> -
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[GitHub] qpid-proton-j pull request #16: PROTON-1938: Fix error propagation in Transp...

2018-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/qpid-proton-j/pull/16


---

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



[jira] [Commented] (PROTON-1938) misaligned Transport set/getErrorCondition and closed() behaviour

2018-10-30 Thread Robbie Gemmell (JIRA)


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

Robbie Gemmell commented on PROTON-1938:


The existing behaviour was a bit more nuanced than described. Calling 
setCondition(ErrorCondition) on the TransportImpl object actually had no 
influence over the ErrorCondition returned from TransportImpl#getCondition() or 
used in certain Close frames at all. It operated on the EndpointImpl superclass 
state whereas the getCondition method was overridden in TransportImpl and only 
operated on a local field in that class which was set inside TransportImpl in 
the closed(TransportException) method, either setting a "connection aborted" 
'framing error' condition if no exception is provided, or a 'framing error' 
with the exception message as its description if an exception was provided.

I have made a different change, removing the extra ErrorCondition field from 
TransportImpl and aligning on use of the parent field used by other objects, 
but retaining as much of the earlier getCondition behaviour of TransportImpl as 
possible/sensible, and ensuring that an ErrorCondition previously set is 
retained in preference to creating a new one when closed(TransportException) is 
called.

> misaligned Transport set/getErrorCondition and closed() behaviour
> -
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Resolved] (PROTON-1938) misaligned Transport set/getErrorCondition and closed() behaviour

2018-10-30 Thread Robbie Gemmell (JIRA)


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

Robbie Gemmell resolved PROTON-1938.

Resolution: Fixed

> misaligned Transport set/getErrorCondition and closed() behaviour
> -
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (PROTON-1938) misaligned Transport set/getErrorCondition and closed() behaviour

2018-10-30 Thread Robbie Gemmell (JIRA)


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

Robbie Gemmell updated PROTON-1938:
---
Summary: misaligned Transport set/getErrorCondition and closed() behaviour  
(was: Transport error condition falsely pretends to be 
ConnectionError.FRAMING_ERROR)

> misaligned Transport set/getErrorCondition and closed() behaviour
> -
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (PROTON-1938) Transport error condition falsely pretends to be ConnectionError.FRAMING_ERROR

2018-10-30 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on PROTON-1938:
-

Commit 19bf22f7e9b88e73db4b195f674d1527dbd713f3 in qpid-proton-j's branch 
refs/heads/master from [~gemmellr]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton-j.git;h=19bf22f ]

PROTON-1938: align TransportImpl set/getCondition behaviour, plus stop the 
closed method replacing an existing ErrorCondition


> Transport error condition falsely pretends to be ConnectionError.FRAMING_ERROR
> --
>
> Key: PROTON-1938
> URL: https://issues.apache.org/jira/browse/PROTON-1938
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-j
>Affects Versions: proton-j-0.29.0
>Reporter: Sreeram Garlapati
>Priority: Minor
> Fix For: proton-j-0.30.0
>
>
> Although at various places in proton library - transport.setCondition() was 
> invoked (for ex: IOHandler.java - invokes several places to communicate 
> socket IOExceptions) - this was overwritten in TransportImpl.closed method 
> and shows up as - ConnectionError.FRAMING_ERROR.
> I created this PR: to help easily identify - where the problem was: 
> [https://github.com/apache/qpid-proton-j/pull/16]
> Please let me know if you need more info or have any inputs finishing the PR 
> at sreer...@microsoft.com...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-8253) [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned from SSLEngine.getSSLParameters().getServerNames()

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-8253:
-
Summary: [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is 
returned from SSLEngine.getSSLParameters().getServerNames()  (was: [Broker-J] 
NPE is thrown in QpidBestFitX509KeyManager when null is returned from 
SSLENgine.getSSLParameters().getServerNames())

> [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned 
> from SSLEngine.getSSLParameters().getServerNames()
> --
>
> Key: QPID-8253
> URL: https://issues.apache.org/jira/browse/QPID-8253
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.7, qpid-java-broker-7.0.6
>Reporter: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7
>
>
> QpidBestFitX509KeyManager does not handle null server names returned from 
> SSLENgine.getSSLParameters().getServerNames()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Created] (QPID-8253) [Broker-J] NPE is thrown in QpidBestFitX509KeyManager when null is returned from SSLENgine.getSSLParameters().getServerNames()

2018-10-30 Thread Alex Rudyy (JIRA)
Alex Rudyy created QPID-8253:


 Summary: [Broker-J] NPE is thrown in QpidBestFitX509KeyManager 
when null is returned from SSLENgine.getSSLParameters().getServerNames()
 Key: QPID-8253
 URL: https://issues.apache.org/jira/browse/QPID-8253
 Project: Qpid
  Issue Type: Bug
  Components: Broker-J
Affects Versions: qpid-java-broker-7.0.6, qpid-java-6.1.7
Reporter: Alex Rudyy
 Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.7


QpidBestFitX509KeyManager does not handle null server names returned from 
SSLENgine.getSSLParameters().getServerNames()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-7885:
-
Description: 
With the Java 9 on the horizon it is time to get Qpid's Java components ready.
* make sure component compile with JDK9
* make sure components run with JRE9

The task of modularization of  broker-j  is excluded from this JIRA, as it 
seems, that many existing modules need to export the same packages which is not 
permitted by Jigsaw module system. Some classes need to be moved into separate 
packages which requires more substantial refactoring.


  was:
With the Java 9 on the horizon it is time to get Qpid's Java components ready.
* make sure component compile with JDK9
* make sure components run with JRE9
* take advantage of the new Java Module System



> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> The task of modularization of  broker-j  is excluded from this JIRA, as it 
> seems, that many existing modules need to export the same packages which is 
> not permitted by Jigsaw module system. Some classes need to be moved into 
> separate packages which requires more substantial refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Assigned] (QPID-7885) [Java Broker] Support Java 9

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy reassigned QPID-7885:


Assignee: Alex Rudyy

> [Java Broker] Support Java 9
> 
>
> Key: QPID-7885
> URL: https://issues.apache.org/jira/browse/QPID-7885
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Lorenz Quack
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> With the Java 9 on the horizon it is time to get Qpid's Java components ready.
> * make sure component compile with JDK9
> * make sure components run with JRE9
> * take advantage of the new Java Module System



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Closed] (QPID-8158) [Broker-J] [System Tests] Refactor BDB HA system tests

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy closed QPID-8158.

Resolution: Fixed

> [Broker-J] [System Tests] Refactor BDB HA system tests
> --
>
> Key: QPID-8158
> URL: https://issues.apache.org/jira/browse/QPID-8158
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J, Java Tests
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> Currently BDB HA system tests are extended from {{QpidBrokerTestCase}}. They 
> need to be refactored to run with {{QpidTestRunner}}  similar to other system 
> tests.
> The BDB HA nodes  should be started as spawn brokers using special  
> {{BrokerAdmin}} allowing to start broker in a separate JVM.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-8245) [Broker-J] [AMQP 0-8..0-91] Decode FieldTable fields on demand

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-8245:
-
Status: Reviewable  (was: In Progress)

> [Broker-J] [AMQP 0-8..0-91] Decode FieldTable fields on demand
> --
>
> Key: QPID-8245
> URL: https://issues.apache.org/jira/browse/QPID-8245
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.1.0
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: 
> 0001-QPID-8245-Broker-J-Stop-rellocating-message-headers-.patch, 
> 0002-QPID-8245-Add-factory-methods-to-create-field-table-.patch, 
> 0003-QPID-8245-Remove-FiledTable-setters-methods-modifyin.patch, 
> 0004-QPID-8245-Decode-field-table-properties-when-require.patch, 
> 0005-QPID-8245-Remove-methods-getXXX-in-order-to-simplify.patch, 
> 0006-QPID-8245-some-code-clean-up.patch, 
> 0007-QPID-8245-Change-decoding-to-decode-only-value-for-t.patch, 
> 0008-QPID-8245-Dispose-QpidByteBuffer-on-decoding-FieldTa.patch
>
>
> At the moment all field table properties are decoded when decode 
> functionality is invoked. For use cases when only some of the field table 
> properties are queried, the decoding functionality can be changed to stop 
> decoding on getting the requested property. Potentially, such approach can 
> improve the performance of routing transient messages when destinations are 
> bound to the routing exchange using selector filters. As filter expression 
> contains only some properties, the decoding of message headers can be stopped 
> on getting all fields specified in selector expression.
> The idea is illustrated by Rob Godfrey on 
> [QPID-8238|https://issues.apache.org/jira/browse/QPID-8238?focusedCommentId=16601936&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16601936]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Assigned] (QPID-8245) [Broker-J] [AMQP 0-8..0-91] Decode FieldTable fields on demand

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy reassigned QPID-8245:


Assignee: Alex Rudyy

> [Broker-J] [AMQP 0-8..0-91] Decode FieldTable fields on demand
> --
>
> Key: QPID-8245
> URL: https://issues.apache.org/jira/browse/QPID-8245
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.1.0
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: 
> 0001-QPID-8245-Broker-J-Stop-rellocating-message-headers-.patch, 
> 0002-QPID-8245-Add-factory-methods-to-create-field-table-.patch, 
> 0003-QPID-8245-Remove-FiledTable-setters-methods-modifyin.patch, 
> 0004-QPID-8245-Decode-field-table-properties-when-require.patch, 
> 0005-QPID-8245-Remove-methods-getXXX-in-order-to-simplify.patch, 
> 0006-QPID-8245-some-code-clean-up.patch, 
> 0007-QPID-8245-Change-decoding-to-decode-only-value-for-t.patch, 
> 0008-QPID-8245-Dispose-QpidByteBuffer-on-decoding-FieldTa.patch
>
>
> At the moment all field table properties are decoded when decode 
> functionality is invoked. For use cases when only some of the field table 
> properties are queried, the decoding functionality can be changed to stop 
> decoding on getting the requested property. Potentially, such approach can 
> improve the performance of routing transient messages when destinations are 
> bound to the routing exchange using selector filters. As filter expression 
> contains only some properties, the decoding of message headers can be stopped 
> on getting all fields specified in selector expression.
> The idea is illustrated by Rob Godfrey on 
> [QPID-8238|https://issues.apache.org/jira/browse/QPID-8238?focusedCommentId=16601936&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16601936]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Assigned] (QPID-8247) [Broker-J] Upgrade mockito to 2.x

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy reassigned QPID-8247:


Assignee: Alex Rudyy

> [Broker-J] Upgrade mockito to 2.x
> -
>
> Key: QPID-8247
> URL: https://issues.apache.org/jira/browse/QPID-8247
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> The Qpid Broker tests depends on mockito version 1.9.5. In order to be able 
> to build broker with java9 and above we need to upgrade mockito at least to 
> version 2.20.0 where compatibility with java 11 was added. Mockito version 
> 2.22.0 is considered to upgrade to.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Updated] (QPID-8247) [Broker-J] Upgrade mockito to 2.x

2018-10-30 Thread Alex Rudyy (JIRA)


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

Alex Rudyy updated QPID-8247:
-
Status: Reviewable  (was: In Progress)

> [Broker-J] Upgrade mockito to 2.x
> -
>
> Key: QPID-8247
> URL: https://issues.apache.org/jira/browse/QPID-8247
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
>
> The Qpid Broker tests depends on mockito version 1.9.5. In order to be able 
> to build broker with java9 and above we need to upgrade mockito at least to 
> version 2.20.0 where compatibility with java 11 was added. Mockito version 
> 2.22.0 is considered to upgrade to.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[jira] [Commented] (DISPATCH-1155) dueling httpRootDirs

2018-10-30 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1155:
--

Github user codecov-io commented on the issue:

https://github.com/apache/qpid-dispatch/pull/409
  
# 
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=h1) 
Report
> Merging 
[#409](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=desc) into 
[master](https://codecov.io/gh/apache/qpid-dispatch/commit/fcf457c981c262d5648d6c1a944059a3c3b47458?src=pr&el=desc)
 will **increase** coverage by `0.02%`.
> The diff coverage is `66.66%`.

[![Impacted file tree 
graph](https://codecov.io/gh/apache/qpid-dispatch/pull/409/graphs/tree.svg?width=650&token=rk2Cgd27pP&height=150&src=pr)](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=tree)

```diff
@@Coverage Diff @@
##   master #409  +/-   ##
==
+ Coverage   85.16%   85.19%   +0.02% 
==
  Files  78   78  
  Lines   1742917430   +1 
==
+ Hits1484314849   +6 
+ Misses   2586 2581   -5
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/connection\_manager.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL2Nvbm5lY3Rpb25fbWFuYWdlci5j)
 | `76.44% <66.66%> (-0.11%)` | :arrow_down: |
| 
[src/router\_core/route\_tables.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX3RhYmxlcy5j)
 | `76.8% <0%> (-0.26%)` | :arrow_down: |
| 
[src/router\_node.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
 | `93.65% <0%> (ø)` | :arrow_up: |
| 
[src/iterator.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL2l0ZXJhdG9yLmM=)
 | `93.66% <0%> (+0.21%)` | :arrow_up: |
| 
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
 | `95.48% <0%> (+0.21%)` | :arrow_up: |
| 
[src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
 | `90.31% <0%> (+0.44%)` | :arrow_up: |
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.84% <0%> (+0.56%)` | :arrow_up: |

--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=continue).
> **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
> Powered by 
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=footer).
 Last update 
[fcf457c...61564b9](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



> dueling httpRootDirs
> 
>
> Key: DISPATCH-1155
> URL: https://issues.apache.org/jira/browse/DISPATCH-1155
> Project: Qpid Dispatch
>  Issue Type: Bug
>Reporter: michael goulish
>Assignee: michael goulish
>Priority: Major
>
> New version of qpid-dispatch-router uses 
> "/usr/share/qpid-dispatch/console/stand-alone" as the default httpRootDir. 
> But when installing new qpid-dispatch-console package, the pages are 
> available at "/usr/share/qpid-dispatch/console".
> This forces the user to define httpRootDir on the listener to bypass this 
> issue.
> Ted suggests this fix:
>  
> Remove the default behavior for httpRootDir. If it is not specified in the 
> configuration for a listener, then HTTP requests shall be rejected on 
> connections to that listener. Such a listener would only be usable for AMQP 
> over websockets.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[GitHub] qpid-dispatch issue #409: DISPATCH-1155 : force explicit config of http root...

2018-10-30 Thread codecov-io
Github user codecov-io commented on the issue:

https://github.com/apache/qpid-dispatch/pull/409
  
# 
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=h1) 
Report
> Merging 
[#409](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=desc) into 
[master](https://codecov.io/gh/apache/qpid-dispatch/commit/fcf457c981c262d5648d6c1a944059a3c3b47458?src=pr&el=desc)
 will **increase** coverage by `0.02%`.
> The diff coverage is `66.66%`.

[![Impacted file tree 
graph](https://codecov.io/gh/apache/qpid-dispatch/pull/409/graphs/tree.svg?width=650&token=rk2Cgd27pP&height=150&src=pr)](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=tree)

```diff
@@Coverage Diff @@
##   master #409  +/-   ##
==
+ Coverage   85.16%   85.19%   +0.02% 
==
  Files  78   78  
  Lines   1742917430   +1 
==
+ Hits1484314849   +6 
+ Misses   2586 2581   -5
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/connection\_manager.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL2Nvbm5lY3Rpb25fbWFuYWdlci5j)
 | `76.44% <66.66%> (-0.11%)` | :arrow_down: |
| 
[src/router\_core/route\_tables.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX3RhYmxlcy5j)
 | `76.8% <0%> (-0.26%)` | :arrow_down: |
| 
[src/router\_node.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
 | `93.65% <0%> (ø)` | :arrow_up: |
| 
[src/iterator.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL2l0ZXJhdG9yLmM=)
 | `93.66% <0%> (+0.21%)` | :arrow_up: |
| 
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
 | `95.48% <0%> (+0.21%)` | :arrow_up: |
| 
[src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
 | `90.31% <0%> (+0.44%)` | :arrow_up: |
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/409/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.84% <0%> (+0.56%)` | :arrow_up: |

--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=continue).
> **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute  (impact)`, `ø = not affected`, `? = missing 
data`
> Powered by 
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=footer).
 Last update 
[fcf457c...61564b9](https://codecov.io/gh/apache/qpid-dispatch/pull/409?src=pr&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



---

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



[jira] [Commented] (DISPATCH-1155) dueling httpRootDirs

2018-10-30 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1155:
--

GitHub user mgoulish opened a pull request:

https://github.com/apache/qpid-dispatch/pull/409

DISPATCH-1155 : force explicit config of http root dir

Force the user to explicit specify HTTP root dir when requesting http 
support on a listener. 
Issue a warning log message if HTTP service is requested, but root dir not 
specified.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mgoulish/qpid-dispatch DISPATCH-1155

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/409.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #409


commit 61564b930f25859702cce2b355d78c30ee3e75cd
Author: Michael Goulish 
Date:   2018-10-30T13:40:23Z

DISPATCH-1155 : force explicit config of http root dir




> dueling httpRootDirs
> 
>
> Key: DISPATCH-1155
> URL: https://issues.apache.org/jira/browse/DISPATCH-1155
> Project: Qpid Dispatch
>  Issue Type: Bug
>Reporter: michael goulish
>Assignee: michael goulish
>Priority: Major
>
> New version of qpid-dispatch-router uses 
> "/usr/share/qpid-dispatch/console/stand-alone" as the default httpRootDir. 
> But when installing new qpid-dispatch-console package, the pages are 
> available at "/usr/share/qpid-dispatch/console".
> This forces the user to define httpRootDir on the listener to bypass this 
> issue.
> Ted suggests this fix:
>  
> Remove the default behavior for httpRootDir. If it is not specified in the 
> configuration for a listener, then HTTP requests shall be rejected on 
> connections to that listener. Such a listener would only be usable for AMQP 
> over websockets.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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



[GitHub] qpid-dispatch pull request #409: DISPATCH-1155 : force explicit config of ht...

2018-10-30 Thread mgoulish
GitHub user mgoulish opened a pull request:

https://github.com/apache/qpid-dispatch/pull/409

DISPATCH-1155 : force explicit config of http root dir

Force the user to explicit specify HTTP root dir when requesting http 
support on a listener. 
Issue a warning log message if HTTP service is requested, but root dir not 
specified.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mgoulish/qpid-dispatch DISPATCH-1155

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/409.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #409


commit 61564b930f25859702cce2b355d78c30ee3e75cd
Author: Michael Goulish 
Date:   2018-10-30T13:40:23Z

DISPATCH-1155 : force explicit config of http root dir




---

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