qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PROTON-1463) Golang AMQP URL incompatable with Qpid Cpp Broker. Node not found: /address when using Qpid C++ Broker
Date Wed, 19 Apr 2017 17:22:41 GMT

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

ASF GitHub Bot commented on PROTON-1463:
----------------------------------------

GitHub user mastash3ff opened a pull request:

    https://github.com/apache/qpid-proton/pull/103

    PROTON-1463 GO-Bindings Added AMQP Address Method

    Original examples used url.Path variable to return the amqp address.  The address returned
added an unnecessary prepended slash character.  Added a amqp.Address() method to filter the
slash character out.  
    
    Original problem running example:  ./receiver.go localhost:5672/example
    Receiver listened on the '/example' address instead of 'example'
    
    Also adjusted the receiver and sender examples to use url.Address() instead of passing
url.Path.

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

    $ git pull https://github.com/mastash3ff/qpid-proton amqpAddressMethod

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

    https://github.com/apache/qpid-proton/pull/103.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 #103
    
----
commit 1c246239789e91666353dd604ce64ce7b368eca3
Author: Brandon Sheffield <brandon>
Date:   2017-04-19T16:50:24Z

    Added Address() function to return an AMQP address.  Previous implementation relied on
URL.Path to
    get an AMQP address.  This had the unintended consequence of returning addresses with
'/' prepended.

commit 7b140d83786b7ed5ee3122a819e7f17f571143a0
Author: Brandon Sheffield <brandon>
Date:   2017-04-19T16:55:43Z

    adjusted Receivers to take address string instead of url.Path string that added an unnecessary
slash character to address

----


> Golang AMQP URL incompatable with Qpid Cpp Broker.  Node not found: /address when using
Qpid C++ Broker
> -------------------------------------------------------------------------------------------------------
>
>                 Key: PROTON-1463
>                 URL: https://issues.apache.org/jira/browse/PROTON-1463
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: go-binding
>    Affects Versions: 0.17.0
>            Reporter: Brandon W Sheffield
>            Assignee: Alan Conway
>
> Go examples not compatible with Qpid C++ broker version 1.36.
> With qpidd running as a daemon, running the go "receive" binary example with no address
will work:
> ./receive localhost:5672
> Listening on 1 connections
> But if provided a address, will get error:
> ./receive localhost:5672/address
> Listening on 1 connections
> 2017/04/18 13:30:54 receive error %v: %vlocalhost:5672/addressamqp:not-found: Node not
found: /address
> Qpid C++ Broker spits out error:
> [Protocol] error Error on attach: Node not found: /address
> Does not behave like this when using the provided Go example broker binary instead of
Qpid C++ broker.  



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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


Mime
View raw message