[jira] [Commented] (DISPATCH-851) Update CMakeLists.txt to build new Dispatch Router Book

2018-06-27 Thread ASF subversion and git services (JIRA)


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

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

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

DISPATCH-851: Correct the cmake logic for adding book targets based on the 
presence of asciidoctor; add asciidoctor to the documented deps and CI config


> Update CMakeLists.txt to build new Dispatch Router Book
> ---
>
> Key: DISPATCH-851
> URL: https://issues.apache.org/jira/browse/DISPATCH-851
> Project: Qpid Dispatch
>  Issue Type: Sub-task
>  Components: Documentation
>Reporter: Ben Hardesty
>Assignee: Justin Ross
>Priority: Major
> Fix For: 1.2.0
>
>
> 1. CMakeLists.txt should build the new Dispatch Router book, currently 
> located in /doc/new-book/.
> 2. The doc/ directory should support multiple books. For example:
> {code}
> /doc/books/ref-guide 
>  
> /doc/books/user-guide
> {code}
> The ref-guide would be for automatically-generated reference material such as 
> management schema and possibly man pages. The user-guide would be for 
> manually-created content (i.e. the Dispatch Router Book).
> 3. The new book should be built using AsciiDoctor instead of AsciiDoc Python 
> (additional info here: 
> http://asciidoctor.org/docs/user-manual/#migrating-from-asciidoc-python).
> Note: For copying the book dir images, note that the images are now located 
> in /doc/new-book/images/. Before, they were located in /doc/book/.



--
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] (DISPATCH-1056) Build fails making docs on python-3-only fedora 28

2018-06-27 Thread Justin Ross (JIRA)


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

Justin Ross reassigned DISPATCH-1056:
-

Assignee: Justin Ross

> Build fails making docs on python-3-only fedora 28
> --
>
> Key: DISPATCH-1056
> URL: https://issues.apache.org/jira/browse/DISPATCH-1056
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
>Reporter: Chuck Rolke
>Assignee: Justin Ross
>Priority: Major
>
> The build failed originally when asciidoc and asciidoctor were not installed. 
> After installing them the build still fail. Proton is build and installed at 
> a shared install prefix: /opt/local
> {{Scanning dependencies of target qdrouterd}}
> {{[ 91%] Building C object router/CMakeFiles/qdrouterd.dir/src/main.c.o}}
> {{[ 93%] Linking C executable qdrouterd}}
> {{[ 93%] Built target qdrouterd}}
> {{Scanning dependencies of target doc_gen}}
> {{[ 95%] Generating qdrouterd_help.adoc}}
> {{[ 96%] Generating qdrouterd.conf.5.adoc}}
> {{[100%] Generating qdmanage_help.adoc}}
> {{[100%] Generating qdstat_help.adoc}}
> {{Traceback (most recent call last):}}
> {{  File "/home/root/git/qpid-dispatch/tools/qdmanage", line 29, in }}
> {{    from qpid_dispatch.management.client import Node, Url}}
> {{  File 
> "/home/root/git/qpid-dispatch/python/qpid_dispatch/management/client.py", 
> line 30, in }}
> {{    import proton}}
> {{ImportError: No module named proton}}
> {{Traceback (most recent call last):}}
> {{  File "/home/root/git/qpid-dispatch/build/run.py", line 123, in }}
> {{    run_path(sys.argv[0], run_name="__main__")}}
> {{  File "/home/root/git/qpid-dispatch/build/run.py", line 107, in run_path}}
> {{    runpy.run_path(file_path, run_name=run_name)}}
> {{  File "/usr/lib64/python2.7/runpy.py", line 252, in run_path}}
> {{    return _run_module_code(code, init_globals, run_name, path_name)}}
> {{  File "/usr/lib64/python2.7/runpy.py", line 82, in _run_module_code}}
> {{    mod_name, mod_fname, mod_loader, pkg_name)}}
> {{  File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code}}
> {{    exec code in run_globals}}
> {{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 68, in 
> }}
> {{    main(sys.argv)}}
> {{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 65, in 
> main}}
> {{    print(help2txt(check_output(program)))}}
> {{  File "/usr/lib64/python2.7/subprocess.py", line 223, in check_output}}
> {{    raise CalledProcessError(retcode, cmd, output=output)}}
> {{subprocess.CalledProcessError: Command 
> '['/home/root/git/qpid-dispatch/tools/qdmanage', '--help']' returned non-zero 
> exit status 1}}
> {{make[2]: *** [docs/man/CMakeFiles/doc_gen.dir/build.make:83: 
> docs/man/qdmanage_help.adoc] Error 1}}
> {{make[2]: *** Deleting file 'docs/man/qdmanage_help.adoc'}}
> {{make[2]: *** Waiting for unfinished jobs}}
> {{Traceback (most recent call last):}}
> {{  File "/home/root/git/qpid-dispatch/tools/qdstat", line 35, in }}
> {{    from qpid_dispatch.management.client import Url, Node, Entity}}
> {{  File 
> "/home/root/git/qpid-dispatch/python/qpid_dispatch/management/client.py", 
> line 30, in }}
> {{    import proton}}
> {{ImportError: No module named proton}}
> {{Traceback (most recent call last):}}
> {{  File "/home/root/git/qpid-dispatch/build/run.py", line 123, in }}
> {{    run_path(sys.argv[0], run_name="__main__")}}
> {{  File "/home/root/git/qpid-dispatch/build/run.py", line 107, in run_path}}
> {{    runpy.run_path(file_path, run_name=run_name)}}
> {{  File "/usr/lib64/python2.7/runpy.py", line 252, in run_path}}
> {{    return _run_module_code(code, init_globals, run_name, path_name)}}
> {{  File "/usr/lib64/python2.7/runpy.py", line 82, in _run_module_code}}
> {{    mod_name, mod_fname, mod_loader, pkg_name)}}
> {{  File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code}}
> {{    exec code in run_globals}}
> {{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 68, in 
> }}
> {{    main(sys.argv)}}
> {{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 65, in 
> main}}
> {{    print(help2txt(check_output(program)))}}
> {{  File "/usr/lib64/python2.7/subprocess.py", line 223, in check_output}}
> {{    raise CalledProcessError(retcode, cmd, output=output)}}
> {{subprocess.CalledProcessError: Command 
> '['/home/root/git/qpid-dispatch/tools/qdstat', '--help']' returned non-zero 
> exit status 1}}
> {{make[2]: *** [docs/man/CMakeFiles/doc_gen.dir/build.make:91: 
> docs/man/qdstat_help.adoc] Error 1}}
> {{make[2]: *** Deleting file 'docs/man/qdstat_help.adoc'}}
> {{make[1]: *** [CMakeFiles/Makefile2:385: 
> docs/man/CMakeFiles/doc_gen.dir/all] Error 2}}
> {{make: *** [Makefile:141: all] Error 2}}
>  



--
This message was sent by Atlassian JIRA

[jira] [Resolved] (DISPATCH-1020) Detach expiring links with closed=true when peer connectivity lost

2018-06-27 Thread Gordon Sim (JIRA)


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

Gordon Sim resolved DISPATCH-1020.
--
Resolution: Fixed

> Detach expiring links with closed=true when peer connectivity lost
> --
>
> Key: DISPATCH-1020
> URL: https://issues.apache.org/jira/browse/DISPATCH-1020
> Project: Qpid Dispatch
>  Issue Type: Improvement
>Reporter: Keith Wall
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.2.0
>
>
> Currently, Dispatch responds to the loss of an underlying connection to a 
> peer by detaching the links established over that connection in the following 
> way.
> {noformat}
> @detach(22) [handle=0, closed=false, error=@error(29) 
> [condition=:"qd:routed-link-lost", description="Connectivity to the peer 
> container was lost”]]
> {noformat}
> As is makes no sense for link defined to expire to attached again, it would 
> be an improvement to the dispatch's behaviour if those links were detached 
> with {{closed=true}}.  This would aid the timely clean-up of resources 
> elsewhere in the AMQP network.



--
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-1020) Detach expiring links with closed=true when peer connectivity lost

2018-06-27 Thread ASF subversion and git services (JIRA)


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

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

Commit 723b73585bec70f009f5409bf29cd1edceaa167d in qpid-dispatch's branch 
refs/heads/master from Gordon Sim
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=723b735 ]

DISPATCH-1020: set close on link lost based on whether terminus would survive 
disconnect


> Detach expiring links with closed=true when peer connectivity lost
> --
>
> Key: DISPATCH-1020
> URL: https://issues.apache.org/jira/browse/DISPATCH-1020
> Project: Qpid Dispatch
>  Issue Type: Improvement
>Reporter: Keith Wall
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.2.0
>
>
> Currently, Dispatch responds to the loss of an underlying connection to a 
> peer by detaching the links established over that connection in the following 
> way.
> {noformat}
> @detach(22) [handle=0, closed=false, error=@error(29) 
> [condition=:"qd:routed-link-lost", description="Connectivity to the peer 
> container was lost”]]
> {noformat}
> As is makes no sense for link defined to expire to attached again, it would 
> be an improvement to the dispatch's behaviour if those links were detached 
> with {{closed=true}}.  This would aid the timely clean-up of resources 
> elsewhere in the AMQP network.



--
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-1056) Build fails making docs on python-3-only fedora 28

2018-06-27 Thread Chuck Rolke (JIRA)
Chuck Rolke created DISPATCH-1056:
-

 Summary: Build fails making docs on python-3-only fedora 28
 Key: DISPATCH-1056
 URL: https://issues.apache.org/jira/browse/DISPATCH-1056
 Project: Qpid Dispatch
  Issue Type: Bug
  Components: Documentation
Affects Versions: 1.1.0
Reporter: Chuck Rolke


The build failed originally when asciidoc and asciidoctor were not installed. 
After installing them the build still fail. Proton is build and installed at a 
shared install prefix: /opt/local

{{Scanning dependencies of target qdrouterd}}
{{[ 91%] Building C object router/CMakeFiles/qdrouterd.dir/src/main.c.o}}
{{[ 93%] Linking C executable qdrouterd}}
{{[ 93%] Built target qdrouterd}}
{{Scanning dependencies of target doc_gen}}
{{[ 95%] Generating qdrouterd_help.adoc}}
{{[ 96%] Generating qdrouterd.conf.5.adoc}}
{{[100%] Generating qdmanage_help.adoc}}
{{[100%] Generating qdstat_help.adoc}}
{{Traceback (most recent call last):}}
{{  File "/home/root/git/qpid-dispatch/tools/qdmanage", line 29, in }}
{{    from qpid_dispatch.management.client import Node, Url}}
{{  File 
"/home/root/git/qpid-dispatch/python/qpid_dispatch/management/client.py", line 
30, in }}
{{    import proton}}
{{ImportError: No module named proton}}
{{Traceback (most recent call last):}}
{{  File "/home/root/git/qpid-dispatch/build/run.py", line 123, in }}
{{    run_path(sys.argv[0], run_name="__main__")}}
{{  File "/home/root/git/qpid-dispatch/build/run.py", line 107, in run_path}}
{{    runpy.run_path(file_path, run_name=run_name)}}
{{  File "/usr/lib64/python2.7/runpy.py", line 252, in run_path}}
{{    return _run_module_code(code, init_globals, run_name, path_name)}}
{{  File "/usr/lib64/python2.7/runpy.py", line 82, in _run_module_code}}
{{    mod_name, mod_fname, mod_loader, pkg_name)}}
{{  File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code}}
{{    exec code in run_globals}}
{{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 68, in 
}}
{{    main(sys.argv)}}
{{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 65, in main}}
{{    print(help2txt(check_output(program)))}}
{{  File "/usr/lib64/python2.7/subprocess.py", line 223, in check_output}}
{{    raise CalledProcessError(retcode, cmd, output=output)}}
{{subprocess.CalledProcessError: Command 
'['/home/root/git/qpid-dispatch/tools/qdmanage', '--help']' returned non-zero 
exit status 1}}
{{make[2]: *** [docs/man/CMakeFiles/doc_gen.dir/build.make:83: 
docs/man/qdmanage_help.adoc] Error 1}}
{{make[2]: *** Deleting file 'docs/man/qdmanage_help.adoc'}}
{{make[2]: *** Waiting for unfinished jobs}}
{{Traceback (most recent call last):}}
{{  File "/home/root/git/qpid-dispatch/tools/qdstat", line 35, in }}
{{    from qpid_dispatch.management.client import Url, Node, Entity}}
{{  File 
"/home/root/git/qpid-dispatch/python/qpid_dispatch/management/client.py", line 
30, in }}
{{    import proton}}
{{ImportError: No module named proton}}
{{Traceback (most recent call last):}}
{{  File "/home/root/git/qpid-dispatch/build/run.py", line 123, in }}
{{    run_path(sys.argv[0], run_name="__main__")}}
{{  File "/home/root/git/qpid-dispatch/build/run.py", line 107, in run_path}}
{{    runpy.run_path(file_path, run_name=run_name)}}
{{  File "/usr/lib64/python2.7/runpy.py", line 252, in run_path}}
{{    return _run_module_code(code, init_globals, run_name, path_name)}}
{{  File "/usr/lib64/python2.7/runpy.py", line 82, in _run_module_code}}
{{    mod_name, mod_fname, mod_loader, pkg_name)}}
{{  File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code}}
{{    exec code in run_globals}}
{{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 68, in 
}}
{{    main(sys.argv)}}
{{  File "/home/root/git/qpid-dispatch/docs/man/help2txt.py", line 65, in main}}
{{    print(help2txt(check_output(program)))}}
{{  File "/usr/lib64/python2.7/subprocess.py", line 223, in check_output}}
{{    raise CalledProcessError(retcode, cmd, output=output)}}
{{subprocess.CalledProcessError: Command 
'['/home/root/git/qpid-dispatch/tools/qdstat', '--help']' returned non-zero 
exit status 1}}
{{make[2]: *** [docs/man/CMakeFiles/doc_gen.dir/build.make:91: 
docs/man/qdstat_help.adoc] Error 1}}
{{make[2]: *** Deleting file 'docs/man/qdstat_help.adoc'}}
{{make[1]: *** [CMakeFiles/Makefile2:385: docs/man/CMakeFiles/doc_gen.dir/all] 
Error 2}}
{{make: *** [Makefile:141: all] Error 2}}

 



--
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 #335: Dispatch 1046 rev2

2018-06-27 Thread kgiusti
GitHub user kgiusti opened a pull request:

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

Dispatch 1046 rev2

Had to change the way we pass args to the host address parse tree api under 
python3

c_char_p is a bytes type under python3.  Passing and returning a unicode 
string causes it to fail.  Had to manually convert between the two.  Also need 
to save a copy of the pattern in the tree as a payload.

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

$ git pull https://github.com/kgiusti/dispatch DISPATCH-1046-REV2

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

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


commit 61f4402807d833ad0c889b7d33fcde8cfdd268f9
Author: Kenneth Giusti 
Date:   2018-06-25T16:05:12Z

DISPATCH-1046: in python3 exceptions no longer have a '.message' attribute

commit 184a62a31049e8a29437bbd3d36e5fec9934fe50
Author: Kenneth Giusti 
Date:   2018-06-26T19:28:04Z

DISPATCH-1046: fix vhost address pattern handling in python3




---

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



[jira] [Commented] (PROTON-1876) expose newer encode/decode operations through the Message interface

2018-06-27 Thread ASF subversion and git services (JIRA)


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

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

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

PROTON-1876 Expose the encode / decode methods using buffers

Expose the encode and decode methods that use WritableBuffer and
ReadableBuffer on the Message interface to allow clients to use newer
no-copy variants of the API easier.

> expose newer encode/decode operations through the Message interface
> ---
>
> Key: PROTON-1876
> URL: https://issues.apache.org/jira/browse/PROTON-1876
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-j
>Reporter: Robbie Gemmell
>Priority: Minor
> Fix For: proton-j-0.28.0
>
>
> The MessageImpl has some newer encode/decode operations that allow for 
> efficiencies over the originals, in terms of reducing buffer copying etc. 
> These should be exposed through the Message interface as well.



--
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-980) Allow address translation on link routes

2018-06-27 Thread Gordon Sim (JIRA)


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

Gordon Sim resolved DISPATCH-980.
-
Resolution: Fixed

> Allow address translation on link routes
> 
>
> Key: DISPATCH-980
> URL: https://issues.apache.org/jira/browse/DISPATCH-980
> Project: Qpid Dispatch
>  Issue Type: New Feature
>Reporter: Gordon Sim
>Assignee: Gordon Sim
>Priority: Major
> Fix For: 1.2.0
>
>
> With auto-links there is an externalAddr field that allows the node on the 
> route-container to have a different name than that the clients see through 
> the router. This is very useful. It would be nice to have a similar feature 
> for link-routes also.



--
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-980) Allow address translation on link routes

2018-06-27 Thread ASF subversion and git services (JIRA)


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

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

Commit 532ebae51a455fba340e9625d71ea62cc64be8f2 in qpid-dispatch's branch 
refs/heads/master from Gordon Sim
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=532ebae ]

DISPATCH-980: add fields to specify prefix to be inserted into- or removed 
from- remote address on link routes


> Allow address translation on link routes
> 
>
> Key: DISPATCH-980
> URL: https://issues.apache.org/jira/browse/DISPATCH-980
> Project: Qpid Dispatch
>  Issue Type: New Feature
>Reporter: Gordon Sim
>Assignee: Gordon Sim
>Priority: Major
> Fix For: 1.2.0
>
>
> With auto-links there is an externalAddr field that allows the node on the 
> route-container to have a different name than that the clients see through 
> the router. This is very useful. It would be nice to have a similar feature 
> for link-routes also.



--
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-1054) Add console test to make test

2018-06-27 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1054:
--

Github user ErnieAllen closed the pull request at:

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


> Add console test to make test
> -
>
> Key: DISPATCH-1054
> URL: https://issues.apache.org/jira/browse/DISPATCH-1054
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Console
>Affects Versions: 1.1.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Minor
>
> Currently the console tests can only be run manually from the 
> console/stand-alone directory.
> Add a CMake system test to run the console tests.



--
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 #333: DISPATCH-1054 add console tests to cmake

2018-06-27 Thread ErnieAllen
Github user ErnieAllen closed the pull request at:

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


---

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



[jira] [Commented] (DISPATCH-1054) Add console test to make test

2018-06-27 Thread ASF subversion and git services (JIRA)


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

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

Commit fcbbb5989540aa5dfab893cd72ce31f48f5776fc in qpid-dispatch's branch 
refs/heads/master from [~eallen]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=fcbbb59 ]

DISPATCH-1054 Add console test to cmake system_tests


> Add console test to make test
> -
>
> Key: DISPATCH-1054
> URL: https://issues.apache.org/jira/browse/DISPATCH-1054
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Console
>Affects Versions: 1.1.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Minor
>
> Currently the console tests can only be run manually from the 
> console/stand-alone directory.
> Add a CMake system test to run the console tests.



--
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-1045) Sometimes close connetion after releasing partial multi-frame messsage

2018-06-27 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1045:
--

Github user ganeshmurthy commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/328#discussion_r198613623
  
--- Diff: src/router_node.c ---
@@ -312,12 +307,30 @@ static void AMQP_rx_handler(void* context, qd_link_t 
*link)
pn_link_name(pn_link));
 }
 
+//
+// The entire message has been received and we are ready to 
consume the delivery by calling pn_link_advance().
+//
+pn_link_advance(pn_link);
+
+//
+// The entire message has been received but this message needs to 
be discarded
+//
+if (qd_message_is_discard(msg)) {
+pn_delivery_update(pnd, qdr_delivery_disposition(delivery));
--- End diff --

Agreed. I have pushed up a new commit to remedy your comment. Please take a 
quick look. Thanks.


> Sometimes close connetion after releasing partial multi-frame messsage
> --
>
> Key: DISPATCH-1045
> URL: https://issues.apache.org/jira/browse/DISPATCH-1045
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Router Node
>Affects Versions: 1.1.0
>Reporter: Alan Conway
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.2.0
>
>
> Since DISPATCH-1012 the router releases undeliverable deliveries.
> In the case of a multi-frame delivery, it is possible for dispatch to release 
> it before the entire delivery has been received. Presently dispatch settles 
> such deliveries and advances its link. That means that if another transfer 
> for the same delivery arrives, dispatch regards it as a new message with an 
> invalid delivery-id and closes the connection. Since the release is 
> asynchronous, there's no way for the client to avoid this possibility.
> Worth checking the spec but I suspect we should be dropping the extra data 
> rather than closing the connection. Need to investigate how this would work 
> in proton - I think we'd update the delivery without settling it and then 
> wait for the remote to settle before finally throwing it away.
>  



--
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 #328: DISPATCH-1045 - Release the delivery only a...

2018-06-27 Thread ganeshmurthy
Github user ganeshmurthy commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/328#discussion_r198613623
  
--- Diff: src/router_node.c ---
@@ -312,12 +307,30 @@ static void AMQP_rx_handler(void* context, qd_link_t 
*link)
pn_link_name(pn_link));
 }
 
+//
+// The entire message has been received and we are ready to 
consume the delivery by calling pn_link_advance().
+//
+pn_link_advance(pn_link);
+
+//
+// The entire message has been received but this message needs to 
be discarded
+//
+if (qd_message_is_discard(msg)) {
+pn_delivery_update(pnd, qdr_delivery_disposition(delivery));
--- End diff --

Agreed. I have pushed up a new commit to remedy your comment. Please take a 
quick look. Thanks.


---

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



[jira] [Updated] (DISPATCH-1055) Flow credit not propagated over link route after drain cycle

2018-06-27 Thread Chuck Rolke (JIRA)


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

Chuck Rolke updated DISPATCH-1055:
--
Attachment: D-2004-direct.html

> Flow credit not propagated over link route after drain cycle
> 
>
> Key: DISPATCH-1055
> URL: https://issues.apache.org/jira/browse/DISPATCH-1055
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Router Node
>Affects Versions: 1.1.0
>Reporter: Chuck Rolke
>Priority: Major
> Attachments: D-2004-direct.html, D-2004-routed.html
>
>
> A transaction test based on activemq-artemis TransactionTest reveals a credit 
> propagation bug. The original case and reproducer code is at 
> [https://issues.jboss.org/browse/ENTMQIC-2004]
>  
> Attached are two traces of the test running direct and routed. I think this 
> is a router bug.
>  * In the direct case (D-2004-direct.html) look at Frames 101-103. Frame 101 
> starts a drain cycle and Frame 102 complete the drain cycle. The the client 
> in Frame 103 immediately sends the receiver 1000 more credits.
>  * In the routed case (D-2004-routed.html) the same sequence is Frames 90-94. 
> Frame 90 (client->router) and Frame 91 (router->broker) start the drain 
> cycle. Frame 92 (broker->router) and Frame 93 (router->client) complete the 
> drain cycle. Then in Frame 94 the client sends 1000 more credits but the 
> router never sends the credits to the broker. That is the bug.
> After Frame 94, where the router does not send any receiver credits to the 
> broker, the broker is unable to send any received messages to the receiver 
> and everything freezes until the test times out.



--
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-1055) Flow credit not propagated over link route after drain cycle

2018-06-27 Thread Chuck Rolke (JIRA)


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

Chuck Rolke updated DISPATCH-1055:
--
Attachment: D-2004-routed.html

> Flow credit not propagated over link route after drain cycle
> 
>
> Key: DISPATCH-1055
> URL: https://issues.apache.org/jira/browse/DISPATCH-1055
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Router Node
>Affects Versions: 1.1.0
>Reporter: Chuck Rolke
>Priority: Major
> Attachments: D-2004-direct.html, D-2004-routed.html
>
>
> A transaction test based on activemq-artemis TransactionTest reveals a credit 
> propagation bug. The original case and reproducer code is at 
> [https://issues.jboss.org/browse/ENTMQIC-2004]
>  
> Attached are two traces of the test running direct and routed. I think this 
> is a router bug.
>  * In the direct case (D-2004-direct.html) look at Frames 101-103. Frame 101 
> starts a drain cycle and Frame 102 complete the drain cycle. The the client 
> in Frame 103 immediately sends the receiver 1000 more credits.
>  * In the routed case (D-2004-routed.html) the same sequence is Frames 90-94. 
> Frame 90 (client->router) and Frame 91 (router->broker) start the drain 
> cycle. Frame 92 (broker->router) and Frame 93 (router->client) complete the 
> drain cycle. Then in Frame 94 the client sends 1000 more credits but the 
> router never sends the credits to the broker. That is the bug.
> After Frame 94, where the router does not send any receiver credits to the 
> broker, the broker is unable to send any received messages to the receiver 
> and everything freezes until the test times out.



--
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-1055) Flow credit not propagated over link route after drain cycle

2018-06-27 Thread Chuck Rolke (JIRA)
Chuck Rolke created DISPATCH-1055:
-

 Summary: Flow credit not propagated over link route after drain 
cycle
 Key: DISPATCH-1055
 URL: https://issues.apache.org/jira/browse/DISPATCH-1055
 Project: Qpid Dispatch
  Issue Type: Bug
  Components: Router Node
Affects Versions: 1.1.0
Reporter: Chuck Rolke


A transaction test based on activemq-artemis TransactionTest reveals a credit 
propagation bug. The original case and reproducer code is at 
[https://issues.jboss.org/browse/ENTMQIC-2004]
 

Attached are two traces of the test running direct and routed. I think this is 
a router bug.
 * In the direct case (D-2004-direct.html) look at Frames 101-103. Frame 101 
starts a drain cycle and Frame 102 complete the drain cycle. The the client in 
Frame 103 immediately sends the receiver 1000 more credits.
 * In the routed case (D-2004-routed.html) the same sequence is Frames 90-94. 
Frame 90 (client->router) and Frame 91 (router->broker) start the drain cycle. 
Frame 92 (broker->router) and Frame 93 (router->client) complete the drain 
cycle. Then in Frame 94 the client sends 1000 more credits but the router never 
sends the credits to the broker. That is the bug.

After Frame 94, where the router does not send any receiver credits to the 
broker, the broker is unable to send any received messages to the receiver and 
everything freezes until the test times out.



--
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 #334: Dispatch 1046 - fix policy handling in python3 env...

2018-06-27 Thread ChugR
Github user ChugR commented on the issue:

https://github.com/apache/qpid-dispatch/pull/334
  
This patch fixes the c_char_p in py3 and improves interfaces with some 
const additions.

The patch also repairs some incorrect handling of policy's parse tree. It 
looks like there were resource leaks when the trees were deleted. I don't think 
that was just a py3 issue.

This PR looks OK to me.


---

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



[jira] [Commented] (DISPATCH-1045) Sometimes close connetion after releasing partial multi-frame messsage

2018-06-27 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1045:
--

Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/328#discussion_r198605509
  
--- Diff: src/router_node.c ---
@@ -312,12 +307,30 @@ static void AMQP_rx_handler(void* context, qd_link_t 
*link)
pn_link_name(pn_link));
 }
 
+//
+// The entire message has been received and we are ready to 
consume the delivery by calling pn_link_advance().
+//
+pn_link_advance(pn_link);
+
+//
+// The entire message has been received but this message needs to 
be discarded
+//
+if (qd_message_is_discard(msg)) {
+pn_delivery_update(pnd, qdr_delivery_disposition(delivery));
--- End diff --

Is the delivery disposition _always_ set when discard is set?  What if the 
disposition was not set?  I think the pn_delivery_update should not be called 
in this case.


> Sometimes close connetion after releasing partial multi-frame messsage
> --
>
> Key: DISPATCH-1045
> URL: https://issues.apache.org/jira/browse/DISPATCH-1045
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Router Node
>Affects Versions: 1.1.0
>Reporter: Alan Conway
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.2.0
>
>
> Since DISPATCH-1012 the router releases undeliverable deliveries.
> In the case of a multi-frame delivery, it is possible for dispatch to release 
> it before the entire delivery has been received. Presently dispatch settles 
> such deliveries and advances its link. That means that if another transfer 
> for the same delivery arrives, dispatch regards it as a new message with an 
> invalid delivery-id and closes the connection. Since the release is 
> asynchronous, there's no way for the client to avoid this possibility.
> Worth checking the spec but I suspect we should be dropping the extra data 
> rather than closing the connection. Need to investigate how this would work 
> in proton - I think we'd update the delivery without settling it and then 
> wait for the remote to settle before finally throwing it away.
>  



--
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 #328: DISPATCH-1045 - Release the delivery only a...

2018-06-27 Thread ted-ross
Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/328#discussion_r198605509
  
--- Diff: src/router_node.c ---
@@ -312,12 +307,30 @@ static void AMQP_rx_handler(void* context, qd_link_t 
*link)
pn_link_name(pn_link));
 }
 
+//
+// The entire message has been received and we are ready to 
consume the delivery by calling pn_link_advance().
+//
+pn_link_advance(pn_link);
+
+//
+// The entire message has been received but this message needs to 
be discarded
+//
+if (qd_message_is_discard(msg)) {
+pn_delivery_update(pnd, qdr_delivery_disposition(delivery));
--- End diff --

Is the delivery disposition _always_ set when discard is set?  What if the 
disposition was not set?  I think the pn_delivery_update should not be called 
in this case.


---

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



[jira] [Commented] (DISPATCH-1054) Add console test to make test

2018-06-27 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1054:
--

Github user codecov-io commented on the issue:

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

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

```diff
@@Coverage Diff @@
##   master #333  +/-   ##
==
- Coverage   86.55%   86.53%   -0.02% 
==
  Files  69   69  
  Lines   1548715489   +2 
==
  Hits1340413404  
- Misses   2083 2085   +2
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/333?src=pr=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
 | `98.88% <0%> (-0.38%)` | :arrow_down: |
| 
[src/message.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL21lc3NhZ2UuYw==)
 | `85.85% <0%> (-0.19%)` | :arrow_down: |
| 
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
 | `95.01% <0%> (-0.12%)` | :arrow_down: |
| 
[src/remote\_sasl.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JlbW90ZV9zYXNsLmM=)
 | `83.28% <0%> (+0.29%)` | :arrow_up: |
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.79% <0%> (+0.57%)` | :arrow_up: |

--

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



> Add console test to make test
> -
>
> Key: DISPATCH-1054
> URL: https://issues.apache.org/jira/browse/DISPATCH-1054
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Console
>Affects Versions: 1.1.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Minor
>
> Currently the console tests can only be run manually from the 
> console/stand-alone directory.
> Add a CMake system test to run the console tests.



--
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 #333: DISPATCH-1054 add console tests to cmake

2018-06-27 Thread codecov-io
Github user codecov-io commented on the issue:

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

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

```diff
@@Coverage Diff @@
##   master #333  +/-   ##
==
- Coverage   86.55%   86.53%   -0.02% 
==
  Files  69   69  
  Lines   1548715489   +2 
==
  Hits1340413404  
- Misses   2083 2085   +2
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/333?src=pr=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
 | `98.88% <0%> (-0.38%)` | :arrow_down: |
| 
[src/message.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL21lc3NhZ2UuYw==)
 | `85.85% <0%> (-0.19%)` | :arrow_down: |
| 
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
 | `95.01% <0%> (-0.12%)` | :arrow_down: |
| 
[src/remote\_sasl.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JlbW90ZV9zYXNsLmM=)
 | `83.28% <0%> (+0.29%)` | :arrow_up: |
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/333/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.79% <0%> (+0.57%)` | :arrow_up: |

--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/333?src=pr=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/333?src=pr=footer).
 Last update 
[e60d6a9...5ff4744](https://codecov.io/gh/apache/qpid-dispatch/pull/333?src=pr=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-1054) Add console test to make test

2018-06-27 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on DISPATCH-1054:
--

GitHub user ErnieAllen opened a pull request:

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

DISPATCH-1054 add console tests to cmake

testing to see if make test successfully runs console tests

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

$ git pull https://github.com/ErnieAllen/qpid-dispatch 
DISPATCH-1054-add-console-test-to-cmake

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

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


commit 5ff474442854babced6a94aa8eb0821532c6ba5a
Author: Ernest Allen 
Date:   2018-06-27T18:09:21Z

DISPATCH-1054 add console tests to cmake




> Add console test to make test
> -
>
> Key: DISPATCH-1054
> URL: https://issues.apache.org/jira/browse/DISPATCH-1054
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Console
>Affects Versions: 1.1.0
>Reporter: Ernest Allen
>Assignee: Ernest Allen
>Priority: Minor
>
> Currently the console tests can only be run manually from the 
> console/stand-alone directory.
> Add a CMake system test to run the console tests.



--
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-1054) Add console test to make test

2018-06-27 Thread Ernest Allen (JIRA)
Ernest Allen created DISPATCH-1054:
--

 Summary: Add console test to make test
 Key: DISPATCH-1054
 URL: https://issues.apache.org/jira/browse/DISPATCH-1054
 Project: Qpid Dispatch
  Issue Type: Improvement
  Components: Console
Affects Versions: 1.1.0
Reporter: Ernest Allen
Assignee: Ernest Allen


Currently the console tests can only be run manually from the 
console/stand-alone directory.

Add a CMake system test to run the console tests.



--
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 #326: Logging

2018-06-27 Thread ted-ross
Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/326#discussion_r198583346
  
--- Diff: src/server.c ---
@@ -651,8 +651,9 @@ static void on_connection_bound(qd_server_t *server, 
pn_event_t *e) {
 pn_sasl_set_allow_insecure_mechs(sasl, 
config->allowInsecureAuthentication);
 sys_mutex_unlock(ctx->server->lock);
 
-qd_log(ctx->server->log_source, QD_LOG_INFO, "Accepted connection 
to %s from %s",
-   name, ctx->rhost_port);
+qd_log(ctx->server->log_source, QD_LOG_INFO,
--- End diff --

There appear to be more arguments than format directives in this call.
Why did you remove the information about the remote host?


---

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



[jira] [Resolved] (DISPATCH-1053) Allow deliveries to be constrained to router-control links by address state

2018-06-27 Thread Ted Ross (JIRA)


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

Ted Ross resolved DISPATCH-1053.

Resolution: Fixed

> Allow deliveries to be constrained to router-control links by address state
> ---
>
> Key: DISPATCH-1053
> URL: https://issues.apache.org/jira/browse/DISPATCH-1053
> Project: Qpid Dispatch
>  Issue Type: Improvement
>  Components: Router Node
>Reporter: Ted Ross
>Assignee: Ted Ross
>Priority: Minor
> Fix For: 1.2.0
>
>
> Add a boolean flag to the address record called 'router_control_only' that, 
> when set, constrains deliveries to that address to only be received via a 
> router-control link.  Use this constraint on the built-in addresses used for 
> inter-router control messages.



--
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-1053) Allow deliveries to be constrained to router-control links by address state

2018-06-27 Thread Ted Ross (JIRA)
Ted Ross created DISPATCH-1053:
--

 Summary: Allow deliveries to be constrained to router-control 
links by address state
 Key: DISPATCH-1053
 URL: https://issues.apache.org/jira/browse/DISPATCH-1053
 Project: Qpid Dispatch
  Issue Type: Improvement
  Components: Router Node
Reporter: Ted Ross
Assignee: Ted Ross
 Fix For: 1.2.0


Add a boolean flag to the address record called 'router_control_only' that, 
when set, constrains deliveries to that address to only be received via a 
router-control link.  Use this constraint on the built-in addresses used for 
inter-router control messages.



--
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-1876) expose newer encode/decode operations through the Message interface

2018-06-27 Thread Robbie Gemmell (JIRA)
Robbie Gemmell created PROTON-1876:
--

 Summary: expose newer encode/decode operations through the Message 
interface
 Key: PROTON-1876
 URL: https://issues.apache.org/jira/browse/PROTON-1876
 Project: Qpid Proton
  Issue Type: Improvement
  Components: proton-j
Reporter: Robbie Gemmell
 Fix For: proton-j-0.28.0


The MessageImpl has some newer encode/decode operations that allow for 
efficiencies over the originals, in terms of reducing buffer copying etc. These 
should be exposed through the Message interface as well.



--
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-1052) minor lock leak in policy code

2018-06-27 Thread Ken Giusti (JIRA)
Ken Giusti created DISPATCH-1052:


 Summary: minor lock leak in policy code
 Key: DISPATCH-1052
 URL: https://issues.apache.org/jira/browse/DISPATCH-1052
 Project: Qpid Dispatch
  Issue Type: Bug
Reporter: Ken Giusti
Assignee: Ken Giusti
 Fix For: 1.2.0


running ctest -VV -R tests_policy with -DUSE_VALGRIND=True:

21: ==17833== 64 bytes in 1 blocks are definitely lost in loss record 1,080 of 
3,621
21: ==17833==at 0x4C2ED47: memalign (vg_replace_malloc.c:857)
21: ==17833==by 0x4C2EE45: posix_memalign (vg_replace_malloc.c:1020)
21: ==17833==by 0x4E706B8: sys_mutex (threading.c:40)
21: ==17833==by 0x4E6C9E8: qd_policy (policy.c:100)
21: ==17833==by 0x4E62B01: qd_dispatch_prepare (dispatch.c:302)
21: ==17833==by 0x10FCAD1D: ffi_call_unix64 (unix64.S:76)
21: ==17833==by 0x10FCA68E: ffi_call (ffi64.c:525)
21: ==17833==by 0x10DB79C4: _call_function_pointer (callproc.c:841)
21: ==17833==by 0x10DB79C4: _ctypes_callproc (callproc.c:1184)
21: ==17833==by 0x10DB168D: PyCFuncPtr_call (_ctypes.c:4077)
21: ==17833==by 0x5BBFBE2: PyObject_Call (abstract.c:2547)
21: ==17833==by 0x5C81991: do_call (ceval.c:4666)
21: ==17833==by 0x5C81991: call_function (ceval.c:4471)
21: ==17833==by 0x5C81991: PyEval_EvalFrameEx (ceval.c:3083)
21: ==17833==by 0x5C86987: PyEval_EvalCodeEx (ceval.c:3681)
21: ==17833== 
21: 




--
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-8214) Reduce the table names size in the JDBC configuration store to fit Oracle's 30 characters limitation

2018-06-27 Thread Olivier VERMEULEN (JIRA)
Olivier VERMEULEN created QPID-8214:
---

 Summary: Reduce the table names size in the JDBC configuration 
store to fit Oracle's 30 characters limitation
 Key: QPID-8214
 URL: https://issues.apache.org/jira/browse/QPID-8214
 Project: Qpid
  Issue Type: Improvement
  Components: Broker-J
Affects Versions: qpid-java-broker-7.0.4
Reporter: Olivier VERMEULEN


'QPID_CONFIGURED_OBJECT_HIERARCHY' is already 32 characters long.

And we should also take into account that users can specify a table name prefix.



--
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-broker-j pull request #9: [QPID-8208] Fix Sybase support for the link-s...

2018-06-27 Thread overmeulen
GitHub user overmeulen opened a pull request:

https://github.com/apache/qpid-broker-j/pull/9

[QPID-8208] Fix Sybase support for the link-store



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

$ git pull https://github.com/overmeulen/qpid-broker-j bugfix/QPID-8208

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

https://github.com/apache/qpid-broker-j/pull/9.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 #9


commit 664c066f00c1c4bb4d381629441d999453735fca
Author: overmeulen 
Date:   2018-06-27T11:57:19Z

[QPID-8208] Fix Sybase support for the link-store




---

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



[jira] [Commented] (QPID-8208) [Broker-J] Improve handling of unexpected exceptions on establishing LDAP connections in SimpleLDAPAuthenticationProvider

2018-06-27 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on QPID-8208:
--

GitHub user overmeulen opened a pull request:

https://github.com/apache/qpid-broker-j/pull/9

[QPID-8208] Fix Sybase support for the link-store



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

$ git pull https://github.com/overmeulen/qpid-broker-j bugfix/QPID-8208

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

https://github.com/apache/qpid-broker-j/pull/9.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 #9


commit 664c066f00c1c4bb4d381629441d999453735fca
Author: overmeulen 
Date:   2018-06-27T11:57:19Z

[QPID-8208] Fix Sybase support for the link-store




> [Broker-J] Improve handling of unexpected exceptions  on establishing LDAP 
> connections in SimpleLDAPAuthenticationProvider
> --
>
> Key: QPID-8208
> URL: https://issues.apache.org/jira/browse/QPID-8208
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, 0.32, qpid-java-6.0, qpid-java-6.0.1, 
> qpid-java-6.0.2, qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, 
> qpid-java-6.1, qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.0.7, qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, 
> qpid-java-broker-7.0.0, qpid-java-6.1.5, qpid-java-broker-7.0.1, 
> qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Priority: Critical
>
> There is a weakness in Qpid exception handling when communication with 
> external services like LDAP. The Broker should take a more defensive approach 
> and handle unexpected exceptions thrown by underlying third-party API in 
> addition to exceptions declared in API interfaces. The unexpected exceptions 
> thrown by underlying API should not affect the stability of the Broker. 
> It was reported that on establishment of connection with LDAP using default 
> context factory {{com.sun.jndi.ldap.LdapCtxFactory}} the creation of  
> {{InitialDirContext}} can end-up in unexpected exception thrown from 
> {{com.sun.jndi.ldap.LdapClient}}. It looks like a defect in 
> {{com.sun.jndi.ldap.LdapClient}}, but I could not find any existing open bug 
> report raised against JVM with similar behaviour. I think that Broker should 
> catch unexpected exception, log it and report authentication failure back to 
> the client.



--
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-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread Keith Wall (JIRA)


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

Keith Wall updated QPID-8213:
-
Priority: Minor  (was: Major)

> [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL) 
> --
>
> Key: QPID-8213
> URL: https://issues.apache.org/jira/browse/QPID-8213
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.0, qpid-java-6.0.1, qpid-java-6.0.2, 
> qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, qpid-java-6.1, 
> qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, qpid-java-6.0.7, 
> qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, qpid-java-broker-7.0.0, 
> qpid-java-6.1.5, 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: Minor
> Fix For: qpid-java-broker-7.1.0
>
>
> A redundant InputStream is opened and not closed in 
> {{SSLUtil#readCertificates(URL)}}



--
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-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread Keith Wall (JIRA)


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

Keith Wall resolved QPID-8213.
--
   Resolution: Fixed
Fix Version/s: qpid-java-broker-7.1.0

> [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL) 
> --
>
> Key: QPID-8213
> URL: https://issues.apache.org/jira/browse/QPID-8213
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.0, qpid-java-6.0.1, qpid-java-6.0.2, 
> qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, qpid-java-6.1, 
> qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, qpid-java-6.0.7, 
> qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, qpid-java-broker-7.0.0, 
> qpid-java-6.1.5, 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
>
>
> A redundant InputStream is opened and not closed in 
> {{SSLUtil#readCertificates(URL)}}



--
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-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread Keith Wall (JIRA)


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

Keith Wall commented on QPID-8213:
--

Change looks good to me. I noticed qpid-jms-amqp-0-x had the same code, albeit 
unused,I removed it with a NO-JIRA commit 
([https://git-wip-us.apache.org/repos/asf?p=qpid-jms-amqp-0-x.git;h=7f67f82ec04a8a346cc64ec1cd824022c9082da0)]
 

> [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL) 
> --
>
> Key: QPID-8213
> URL: https://issues.apache.org/jira/browse/QPID-8213
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.0, qpid-java-6.0.1, qpid-java-6.0.2, 
> qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, qpid-java-6.1, 
> qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, qpid-java-6.0.7, 
> qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, qpid-java-broker-7.0.0, 
> qpid-java-6.1.5, 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
>
> A redundant InputStream is opened and not closed in 
> {{SSLUtil#readCertificates(URL)}}



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



Re: InputStream in SSLUtil.java line 449

2018-06-27 Thread Oleksandr Rudyy
Hi Jesse,

Thanks for letting us know about the defect.

I fixed the defect as part of QPID-8213.

Kind Regards,
Alex

[1] https://issues.apache.org/jira/browse/QPID-8213

On 27 June 2018 at 02:27, Jesse  wrote:
> Hello,
>
> I noticed that there may be an unclosed resource around line 449 of
> org.apache.qpid.server.transport.network.security.ssl.SSLUtil.java
>
> Let me know what you think or if you need more details.
>
> Thanks,
>
> Jesse
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
> For additional commands, e-mail: dev-h...@qpid.apache.org
>

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



[jira] [Updated] (QPID-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread Alex Rudyy (JIRA)


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

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

> [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL) 
> --
>
> Key: QPID-8213
> URL: https://issues.apache.org/jira/browse/QPID-8213
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.0, qpid-java-6.0.1, qpid-java-6.0.2, 
> qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, qpid-java-6.1, 
> qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, qpid-java-6.0.7, 
> qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, qpid-java-broker-7.0.0, 
> qpid-java-6.1.5, 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
>
> A redundant InputStream is opened and not closed in 
> {{SSLUtil#readCertificates(URL)}}



--
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] [Comment Edited] (PROTON-1862) Idle timeout not working on Linux

2018-06-27 Thread Jeremy (JIRA)


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

Jeremy edited comment on PROTON-1862 at 6/27/18 9:50 AM:
-

Any updates as to when can this issue be addressed?

I can work on a patch, however, I need your validation before, on whether or 
not this is indeed a regression.


was (Author: jeremy.aouad):
Any updates as to when can this issue be addressed?

I can work on a patch, however, I need your validation before if this is indeed 
a regression.

> Idle timeout not working on Linux
> -
>
> Key: PROTON-1862
> URL: https://issues.apache.org/jira/browse/PROTON-1862
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: cpp-binding
>Affects Versions: proton-c-0.22.0
>Reporter: Jeremy
>Priority: Critical
>  Labels: reproducer
> Attachments: test_case.cpp
>
>
> We faced an issue with the idle timeout on linux. On windows, it seems to 
> work.
> In our proton feature test suite, we test the idle timeout feature by doing a 
> sleep in the method on_session_open.
> This should trigger a connection timeout. It works on windows, and it used to 
> work with proton v0.16.0 on windows and linux.
> Removing the sleep from the on_session_open and putting it in 
> on_connection_open, yields the same result.
> See attached file to reproduce.
> Machines:
>  * Windows machine
>  ** OS: Windows 7
>  ** Compiler: MSVC 2013 Version 12 Update 5
>  * Linux machine
>  ** OS: Red Hat Enterprise Linux Server release 6.4 (Santiago)
>  ** Compiler: g++491 (GCC) 4.9.2 20150212 (Red Hat 4.9.2-6)



--
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-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread Alex Rudyy (JIRA)


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

Alex Rudyy reassigned QPID-8213:


Assignee: Alex Rudyy

> [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL) 
> --
>
> Key: QPID-8213
> URL: https://issues.apache.org/jira/browse/QPID-8213
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.0, qpid-java-6.0.1, qpid-java-6.0.2, 
> qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, qpid-java-6.1, 
> qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, qpid-java-6.0.7, 
> qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, qpid-java-broker-7.0.0, 
> qpid-java-6.1.5, 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
>
> A redundant InputStream is opened and not closed in 
> {{SSLUtil#readCertificates(URL)}}



--
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-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread ASF subversion and git services (JIRA)


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

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

Commit c018e1ac9d21e9f5eb38d2ae7a26a31e63c07fdf 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=c018e1a ]

QPID-8213: [Broker-J] Do not open redundant InputStream in 
SSLUtil#readCertificates


> [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL) 
> --
>
> Key: QPID-8213
> URL: https://issues.apache.org/jira/browse/QPID-8213
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.0, qpid-java-6.0.1, qpid-java-6.0.2, 
> qpid-java-6.0.3, qpid-java-6.0.4, qpid-java-6.0.5, qpid-java-6.1, 
> qpid-java-6.0.6, qpid-java-6.1.1, qpid-java-6.1.2, qpid-java-6.0.7, 
> qpid-java-6.1.3, qpid-java-6.0.8, qpid-java-6.1.4, qpid-java-broker-7.0.0, 
> qpid-java-6.1.5, 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
>Priority: Major
>
> A redundant InputStream is opened and not closed in 
> {{SSLUtil#readCertificates(URL)}}



--
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-1862) Idle timeout not working on Linux

2018-06-27 Thread Jeremy (JIRA)


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

Jeremy commented on PROTON-1862:


Any updates as to when can this issue be addressed?

I can work on a patch, however, I need your validation before if this is indeed 
a regression.

> Idle timeout not working on Linux
> -
>
> Key: PROTON-1862
> URL: https://issues.apache.org/jira/browse/PROTON-1862
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: cpp-binding
>Affects Versions: proton-c-0.22.0
>Reporter: Jeremy
>Priority: Critical
>  Labels: reproducer
> Attachments: test_case.cpp
>
>
> We faced an issue with the idle timeout on linux. On windows, it seems to 
> work.
> In our proton feature test suite, we test the idle timeout feature by doing a 
> sleep in the method on_session_open.
> This should trigger a connection timeout. It works on windows, and it used to 
> work with proton v0.16.0 on windows and linux.
> Removing the sleep from the on_session_open and putting it in 
> on_connection_open, yields the same result.
> See attached file to reproduce.
> Machines:
>  * Windows machine
>  ** OS: Windows 7
>  ** Compiler: MSVC 2013 Version 12 Update 5
>  * Linux machine
>  ** OS: Red Hat Enterprise Linux Server release 6.4 (Santiago)
>  ** Compiler: g++491 (GCC) 4.9.2 20150212 (Red Hat 4.9.2-6)



--
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-8213) [Broker-J] InputStream is not closed in SSLUtil#readCertificates(URL)

2018-06-27 Thread Alex Rudyy (JIRA)
Alex Rudyy created QPID-8213:


 Summary: [Broker-J] InputStream is not closed in 
SSLUtil#readCertificates(URL) 
 Key: QPID-8213
 URL: https://issues.apache.org/jira/browse/QPID-8213
 Project: Qpid
  Issue Type: Bug
  Components: Broker-J
Affects Versions: qpid-java-broker-7.0.6, qpid-java-broker-7.0.5, 
qpid-java-broker-7.0.4, qpid-java-broker-7.0.1, qpid-java-6.1.5, 
qpid-java-broker-7.0.0, qpid-java-6.1.4, qpid-java-6.0.8, qpid-java-6.1.3, 
qpid-java-6.0.7, qpid-java-6.1.2, qpid-java-6.1.1, qpid-java-6.0.6, 
qpid-java-6.1, qpid-java-6.0.5, qpid-java-6.0.4, qpid-java-6.0.3, 
qpid-java-6.0.2, qpid-java-6.0.1, qpid-java-6.0, qpid-java-broker-7.0.2, 
qpid-java-broker-7.0.3, qpid-java-6.1.6
Reporter: Alex Rudyy


A redundant InputStream is opened and not closed in 
{{SSLUtil#readCertificates(URL)}}



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