[jira] [Updated] (DISPATCH-1147) Expose address priority

2018-10-29 Thread Ganesh Murthy (JIRA)


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

Ganesh Murthy updated DISPATCH-1147:

Fix Version/s: (was: 1.4.1)
   1.5.0

> Expose address priority
> ---
>
> Key: DISPATCH-1147
> URL: https://issues.apache.org/jira/browse/DISPATCH-1147
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Management Agent, Tests, Tools
>Affects Versions: 1.4.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Major
> Fix For: 1.5.0
>
>
> Expose the new address priority to management calls.
> Modify qdstat -a to show priority
> Add a test to ensure qdstat -a outputs address priority



--
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-1158) Add background map to console's topology page

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


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

ASF GitHub Bot commented on DISPATCH-1158:
--

Github user codecov-io commented on the issue:

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

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

```diff
@@Coverage Diff @@
##   master #408  +/-   ##
==
+ Coverage   85.16%   85.24%   +0.08% 
==
  Files  78   78  
  Lines   1742917429  
==
+ Hits1484314858  +15 
+ Misses   2586 2571  -15
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/408?src=pr&el=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/parse.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3BhcnNlLmM=)
 | `85.25% <0%> (-0.27%)` | :arrow_down: |
| 
[src/router\_core/route\_tables.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/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/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
 | `93.78% <0%> (+0.12%)` | :arrow_up: |
| 
[src/container.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL2NvbnRhaW5lci5j)
 | `76.96% <0%> (+0.19%)` | :arrow_up: |
| 
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
 | `95.48% <0%> (+0.21%)` | :arrow_up: |
| 
[src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
 | `93.42% <0%> (+0.28%)` | :arrow_up: |
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.84% <0%> (+0.56%)` | :arrow_up: |
| 
[...c/router\_core/modules/test\_hooks/core\_test\_hooks.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL21vZHVsZXMvdGVzdF9ob29rcy9jb3JlX3Rlc3RfaG9va3MuYw==)
 | `93.54% <0%> (+0.96%)` | :arrow_up: |
| 
[src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
 | `90.9% <0%> (+1.04%)` | :arrow_up: |
| 
[src/router\_core/core\_timer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2NvcmVfdGltZXIuYw==)
 | `96.42% <0%> (+1.78%)` | :arrow_up: |

--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/408?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/408?src=pr&el=footer).
 Last update 
[fcf457c...790bd3e](https://codecov.io/gh/apache/qpid-dispatch/pull/408?src=pr&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



> Add background map to console's topology page
> -
>
> Key: DISPATCH-1158
> URL: https://issues.apache.org/jira/browse/DISPATCH-1158
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Console
>Affects Versions: 1.4.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Major
>
> It should:
>  * show a world map
>  * allow panning
>  * wrap (if a country is panned off one side it should show up on other side)
>  * allow zooming in/out
>  * pin routers to a geo location (if a router is moved to a geographic 
> location, it should stay there when the map is panned/zoomed)
>  * remember router's pinned location
>  * remember pan and zoom settings
>  * allow changing of land and ocean 

[GitHub] qpid-dispatch issue #408: ernie-DISPATCH-1158 Add background map to console'...

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

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

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

```diff
@@Coverage Diff @@
##   master #408  +/-   ##
==
+ Coverage   85.16%   85.24%   +0.08% 
==
  Files  78   78  
  Lines   1742917429  
==
+ Hits1484314858  +15 
+ Misses   2586 2571  -15
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/408?src=pr&el=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/parse.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3BhcnNlLmM=)
 | `85.25% <0%> (-0.27%)` | :arrow_down: |
| 
[src/router\_core/route\_tables.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/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/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
 | `93.78% <0%> (+0.12%)` | :arrow_up: |
| 
[src/container.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL2NvbnRhaW5lci5j)
 | `76.96% <0%> (+0.19%)` | :arrow_up: |
| 
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
 | `95.48% <0%> (+0.21%)` | :arrow_up: |
| 
[src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
 | `93.42% <0%> (+0.28%)` | :arrow_up: |
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.84% <0%> (+0.56%)` | :arrow_up: |
| 
[...c/router\_core/modules/test\_hooks/core\_test\_hooks.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL21vZHVsZXMvdGVzdF9ob29rcy9jb3JlX3Rlc3RfaG9va3MuYw==)
 | `93.54% <0%> (+0.96%)` | :arrow_up: |
| 
[src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
 | `90.9% <0%> (+1.04%)` | :arrow_up: |
| 
[src/router\_core/core\_timer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/408/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2NvcmVfdGltZXIuYw==)
 | `96.42% <0%> (+1.78%)` | :arrow_up: |

--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/408?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/408?src=pr&el=footer).
 Last update 
[fcf457c...790bd3e](https://codecov.io/gh/apache/qpid-dispatch/pull/408?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



[GitHub] qpid-dispatch pull request #408: ernie-DISPATCH-1158 Add background map to c...

2018-10-29 Thread ErnieAllen
GitHub user ErnieAllen opened a pull request:

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

ernie-DISPATCH-1158 Add background map to console's topology page

New feature for demos: optional background map on topology page.

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

$ git pull https://github.com/ErnieAllen/qpid-dispatch ernie-DISPATCH-1158

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

https://github.com/apache/qpid-dispatch/pull/408.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 #408






---

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



[jira] [Commented] (DISPATCH-1158) Add background map to console's topology page

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


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

ASF GitHub Bot commented on DISPATCH-1158:
--

GitHub user ErnieAllen opened a pull request:

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

ernie-DISPATCH-1158 Add background map to console's topology page

New feature for demos: optional background map on topology page.

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

$ git pull https://github.com/ErnieAllen/qpid-dispatch ernie-DISPATCH-1158

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

https://github.com/apache/qpid-dispatch/pull/408.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 #408






> Add background map to console's topology page
> -
>
> Key: DISPATCH-1158
> URL: https://issues.apache.org/jira/browse/DISPATCH-1158
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Console
>Affects Versions: 1.4.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Major
>
> It should:
>  * show a world map
>  * allow panning
>  * wrap (if a country is panned off one side it should show up on other side)
>  * allow zooming in/out
>  * pin routers to a geo location (if a router is moved to a geographic 
> location, it should stay there when the map is panned/zoomed)
>  * remember router's pinned location
>  * remember pan and zoom settings
>  * allow changing of land and ocean colors
>  *



--
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-8242) [Broker-J] JDBC store should remove message content/metadata asynchronously

2018-10-29 Thread Rob Godfrey (JIRA)


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

Rob Godfrey reassigned QPID-8242:
-

Assignee: Alex Rudyy  (was: Rob Godfrey)

> [Broker-J] JDBC store should remove message content/metadata asynchronously
> ---
>
> Key: QPID-8242
> URL: https://issues.apache.org/jira/browse/QPID-8242
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Rob Godfrey
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: QPID-8242.patch
>
>
> Currently in the JDBC store the message content is deleted synchronously 
> after the last reference is removed.  There is no need to make this operation 
> synchronous and the message deletes could be batched.



--
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-8242) [Broker-J] JDBC store should remove message content/metadata asynchronously

2018-10-29 Thread Rob Godfrey (JIRA)


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

Rob Godfrey updated QPID-8242:
--
Status: Reviewable  (was: In Progress)

> [Broker-J] JDBC store should remove message content/metadata asynchronously
> ---
>
> Key: QPID-8242
> URL: https://issues.apache.org/jira/browse/QPID-8242
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Rob Godfrey
>Assignee: Rob Godfrey
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: QPID-8242.patch
>
>
> Currently in the JDBC store the message content is deleted synchronously 
> after the last reference is removed.  There is no need to make this operation 
> synchronous and the message deletes could be batched.



--
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-8242) [Broker-J] JDBC store should remove message content/metadata asynchronously

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


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

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

Commit bb57b51593128b31c796f80cba249f9a3e126177 in qpid-broker-j's branch 
refs/heads/master from rgodfrey
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=bb57b51 ]

QPID-8242 : JDBC store should remove message content/metadata asynchronously


> [Broker-J] JDBC store should remove message content/metadata asynchronously
> ---
>
> Key: QPID-8242
> URL: https://issues.apache.org/jira/browse/QPID-8242
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Rob Godfrey
>Assignee: Rob Godfrey
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: QPID-8242.patch
>
>
> Currently in the JDBC store the message content is deleted synchronously 
> after the last reference is removed.  There is no need to make this operation 
> synchronous and the message deletes could be batched.



--
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-8242) [Broker-J] JDBC store should remove message content/metadata asynchronously

2018-10-29 Thread Rob Godfrey (JIRA)


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

Rob Godfrey reassigned QPID-8242:
-

Assignee: Rob Godfrey

> [Broker-J] JDBC store should remove message content/metadata asynchronously
> ---
>
> Key: QPID-8242
> URL: https://issues.apache.org/jira/browse/QPID-8242
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Rob Godfrey
>Assignee: Rob Godfrey
>Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments: QPID-8242.patch
>
>
> Currently in the JDBC store the message content is deleted synchronously 
> after the last reference is removed.  There is no need to make this operation 
> synchronous and the message deletes could be batched.



--
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] (QPIDJMS-423) Log only connection URI in the connection initialized event handler

2018-10-29 Thread Timothy Bish (JIRA)


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

Timothy Bish resolved QPIDJMS-423.
--
Resolution: Fixed

Used the built in URI tools to remove any query options from the connect string 
as it handles the variations in URIs we support for added caution. 

> Log only connection URI in the connection initialized event handler
> ---
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI with all connection configuration options instead just the URI 
> portion should be logged with the query portion omitted.



--
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] (QPIDJMS-423) Log only connection URI in the connection initialized event handler

2018-10-29 Thread Timothy Bish (JIRA)


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

Timothy Bish updated QPIDJMS-423:
-
Fix Version/s: 0.38.0

> Log only connection URI in the connection initialized event handler
> ---
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
> Fix For: 0.38.0
>
>
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI with all connection configuration options instead just the URI 
> portion should be logged with the query portion omitted.



--
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-1158) Add background map to console's topology page

2018-10-29 Thread Ernest Allen (JIRA)
Ernest Allen created DISPATCH-1158:
--

 Summary: Add background map to console's topology page
 Key: DISPATCH-1158
 URL: https://issues.apache.org/jira/browse/DISPATCH-1158
 Project: Qpid Dispatch
  Issue Type: Improvement
  Components: Console
Affects Versions: 1.4.0
Reporter: Ernest Allen
Assignee: Ernest Allen


It should:
 * show a world map
 * allow panning
 * wrap (if a country is panned off one side it should show up on other side)
 * allow zooming in/out
 * pin routers to a geo location (if a router is moved to a geographic 
location, it should stay there when the map is panned/zoomed)
 * remember router's pinned location
 * remember pan and zoom settings
 * allow changing of land and ocean colors
 *



--
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-423) Log only connection URI in the connection initialized event handler

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


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

ASF GitHub Bot commented on QPIDJMS-423:


Github user asfgit closed the pull request at:

https://github.com/apache/qpid-jms/pull/23


> Log only connection URI in the connection initialized event handler
> ---
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI with all connection configuration options instead just the URI 
> portion should be logged with the query portion omitted.



--
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-jms pull request #23: QPIDJMS-423 onConnectionEstablished(): log only m...

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

https://github.com/apache/qpid-jms/pull/23


---

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



[jira] [Commented] (QPIDJMS-423) Log only connection URI in the connection initialized event handler

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


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

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

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

QPIDJMS-423 Log only the host and port data from the URI on connect

Update the connection event handler to log only the on-query portion of
the URI when processing the event.

This closes #23

> Log only connection URI in the connection initialized event handler
> ---
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI with all connection configuration options instead just the URI 
> portion should be logged with the query portion omitted.



--
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] (QPIDJMS-423) Log only connection URI in the connection initialized event handler

2018-10-29 Thread Timothy Bish (JIRA)


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

Timothy Bish updated QPIDJMS-423:
-
Description: The method JmsConnection.onConnectionEstablished(final URI 
remoteURI) logs this URI with all connection configuration options instead just 
the URI portion should be logged with the query portion omitted.  (was: The 
broker URI may contain sensitive info (like path to trust / key stores, and 
related *passwords*), and this info is being logged.

Sample:
{code:xml}



{code}

The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs this 
URI as is, therefore disclosing some passwords.

Only essential info just be logged, i.e. scheme, host and port.)

> Log only connection URI in the connection initialized event handler
> ---
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI with all connection configuration options instead just the URI 
> portion should be logged with the query portion omitted.



--
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] (QPIDJMS-423) Log only connection URI in the connection initialized event handler

2018-10-29 Thread Timothy Bish (JIRA)


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

Timothy Bish updated QPIDJMS-423:
-
Summary: Log only connection URI in the connection initialized event 
handler  (was: Log only connection URI in the connection initialized )

> Log only connection URI in the connection initialized event handler
> ---
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The broker URI may contain sensitive info (like path to trust / key stores, 
> and related *passwords*), and this info is being logged.
> Sample:
> {code:xml}
>  class="org.apache.qpid.jms.JmsConnectionFactory">
> 
> 
> {code}
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI as is, therefore disclosing some passwords.
> Only essential info just be logged, i.e. scheme, host and port.



--
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] (QPIDJMS-423) Log only connection URI in the connection initialized

2018-10-29 Thread Timothy Bish (JIRA)


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

Timothy Bish updated QPIDJMS-423:
-
Summary: Log only connection URI in the connection initialized   (was: 
Avoid disclosing sensitive info when logging Remote Broker URI)

> Log only connection URI in the connection initialized 
> --
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The broker URI may contain sensitive info (like path to trust / key stores, 
> and related *passwords*), and this info is being logged.
> Sample:
> {code:xml}
>  class="org.apache.qpid.jms.JmsConnectionFactory">
> 
> 
> {code}
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI as is, therefore disclosing some passwords.
> Only essential info just be logged, i.e. scheme, host and port.



--
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] (PROTON-1960) [Go Electron] Unable to send a message larger than 16k in size

2018-10-29 Thread Rob KIng (JIRA)
Rob KIng created PROTON-1960:


 Summary: [Go Electron] Unable to send a message larger than 16k in 
size
 Key: PROTON-1960
 URL: https://issues.apache.org/jira/browse/PROTON-1960
 Project: Qpid Proton
  Issue Type: Bug
Reporter: Rob KIng


Unable to send a message larger than 16k in size from a sending program using 
the Go electron bindings to the qpidd broker. Attempting to send a message of 
this size causes the program to hang in the electron package. Some discussion 
and some debug output from the broker can be found at 
[http://qpid.2158936.n2.nabble.com/Go-Electron-Large-Messages-Fail-To-Send-td7680219.html]

Issue can be recreated using a slightly modified version of send.go in the 
electron package [https://play.golang.org/p/2dQ4jdlKHKi] sending to qpidd 
version 1.38



--
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-423) Avoid disclosing sensitive info when logging Remote Broker URI

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


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

ASF GitHub Bot commented on QPIDJMS-423:


Github user benoitdevos commented on the issue:

https://github.com/apache/qpid-jms/pull/23
  
@tabish121 I created 
[QPIDJMS-423](https://issues.apache.org/jira/browse/QPIDJMS-423) and changed 
the commit message.


> Avoid disclosing sensitive info when logging Remote Broker URI
> --
>
> Key: QPIDJMS-423
> URL: https://issues.apache.org/jira/browse/QPIDJMS-423
> Project: Qpid JMS
>  Issue Type: Improvement
>  Components: qpid-jms-client
>Affects Versions: 0.37.0
>Reporter: Benoit Devos
>Priority: Minor
>
> The broker URI may contain sensitive info (like path to trust / key stores, 
> and related *passwords*), and this info is being logged.
> Sample:
> {code:xml}
>  class="org.apache.qpid.jms.JmsConnectionFactory">
> 
> 
> {code}
> The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs 
> this URI as is, therefore disclosing some passwords.
> Only essential info just be logged, i.e. scheme, host and port.



--
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-jms issue #23: QPIDJMS-423 onConnectionEstablished(): log only main bro...

2018-10-29 Thread benoitdevos
Github user benoitdevos commented on the issue:

https://github.com/apache/qpid-jms/pull/23
  
@tabish121 I created 
[QPIDJMS-423](https://issues.apache.org/jira/browse/QPIDJMS-423) and changed 
the commit message.


---

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



[jira] [Created] (QPIDJMS-423) Avoid disclosing sensitive info when logging Remote Broker URI

2018-10-29 Thread Benoit Devos (JIRA)
Benoit Devos created QPIDJMS-423:


 Summary: Avoid disclosing sensitive info when logging Remote 
Broker URI
 Key: QPIDJMS-423
 URL: https://issues.apache.org/jira/browse/QPIDJMS-423
 Project: Qpid JMS
  Issue Type: Improvement
  Components: qpid-jms-client
Affects Versions: 0.37.0
Reporter: Benoit Devos


The broker URI may contain sensitive info (like path to trust / key stores, and 
related *passwords*), and this info is being logged.

Sample:
{code:xml}



{code}

The method JmsConnection.onConnectionEstablished(final URI remoteURI) logs this 
URI as is, therefore disclosing some passwords.

Only essential info just be logged, i.e. scheme, host and port.



--
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