[jira] [Commented] (PROTON-1135) [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default
[ https://issues.apache.org/jira/browse/PROTON-1135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15228410#comment-15228410 ] Andrew Stitcher commented on PROTON-1135: - This is fixed by no longer sending pipelined frames from SASL into AMQP frames. I've modified the tests so that we are still testing that proton-c correctly handles incoming pipelined frames (by generating the frames by hand and hardcoding them in the test). So this should ensure that we do carry on accepting pipelined frames. > [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default > - > > Key: PROTON-1135 > URL: https://issues.apache.org/jira/browse/PROTON-1135 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c >Affects Versions: 0.12.0 >Reporter: Ganesh Murthy >Assignee: Andrew Stitcher > > Dispatch router (which uses Proton-c) currently sends pipelined SASL and OPEN > frames by default when connecting out to other peers using the ANONYMOUS > mech, as shown in the following trace - > {code} > [0x7f41f80079c0]: -> SASL > [0x7f41f80079c0]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, > initial-response=b"anonymous@localhost.localdomain"] > [0x7f41f80079c0]: -> AMQP > [0x7f41f80079c0]:0 -> @open(16) [container-id="Qpid.Dispatch.Router.A", > max-frame-size=65536, channel-max=32767, idle-time-out=8000, > offered-capabilities=:"ANONYMOUS-RELAY", > properties={:product="qpid-dispatch-router", :version="0.6.0"}] > [0x7f41f80079c0]: <- SASL > [0x7f41f80079c0]:0 <- @sasl-mechanisms(64) [sasl-server-mechanisms=:ANONYMOUS] > [0x7f41f80079c0]:0 <- @sasl-outcome(68) [code=0] > [0x7f41f80079c0]: <- AMQP > [0x7f41f80079c0]:0 <- @open(16) > [container-id="ce103199-af03-4d37-bb35-24ad4e55653e", channel-max=32767, > idle-time-out=8000, offered-capabilities=@PN_SYMBOL[:"ANONYMOUS-RELAY"], > properties={:product="qpid-cpp", :version="0.35", :platform="Linux", > :host="localhost.localdomain"}] > {code} > The AMQP 1.0 spec does not make it clear that this is supported (e.g. see > diagram below) but in any case various components have shown difficulty with > it (such as PROTON-1135 just raised, and QPID-6639 which has yet to be > included in a release but permitted the above protocol trace log). > {code} > TCP Client TCP Server > = > AMQP%d3.1.0.0 -> > <- AMQP%d3.1.0.0 > : > : > > : > : > AMQP%d0.1.0.0 -> > (over SASL secured connection) > <- AMQP%d0.1.0.0 > open -> > <- open > {code} > Proton should by default disable sending pipelined OPEN frames for ANONYMOUS > logins, to aid compatibility with other components that don't expect/handle > such behaviour. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (PROTON-1135) [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default
[ https://issues.apache.org/jira/browse/PROTON-1135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15228378#comment-15228378 ] ASF subversion and git services commented on PROTON-1135: - Commit 6738c16ffcb93981f1bb936b21e86515b1787f54 in qpid-proton's branch refs/heads/master from [~astitcher] [ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=6738c16 ] PROTON-1135: Do not pipeline SASL frames into AMQP frames from client side - Test that we will accept pipelined frames though. > [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default > - > > Key: PROTON-1135 > URL: https://issues.apache.org/jira/browse/PROTON-1135 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c >Affects Versions: 0.12.0 >Reporter: Ganesh Murthy >Assignee: Andrew Stitcher > > Dispatch router (which uses Proton-c) currently sends pipelined SASL and OPEN > frames by default when connecting out to other peers using the ANONYMOUS > mech, as shown in the following trace - > {code} > [0x7f41f80079c0]: -> SASL > [0x7f41f80079c0]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, > initial-response=b"anonymous@localhost.localdomain"] > [0x7f41f80079c0]: -> AMQP > [0x7f41f80079c0]:0 -> @open(16) [container-id="Qpid.Dispatch.Router.A", > max-frame-size=65536, channel-max=32767, idle-time-out=8000, > offered-capabilities=:"ANONYMOUS-RELAY", > properties={:product="qpid-dispatch-router", :version="0.6.0"}] > [0x7f41f80079c0]: <- SASL > [0x7f41f80079c0]:0 <- @sasl-mechanisms(64) [sasl-server-mechanisms=:ANONYMOUS] > [0x7f41f80079c0]:0 <- @sasl-outcome(68) [code=0] > [0x7f41f80079c0]: <- AMQP > [0x7f41f80079c0]:0 <- @open(16) > [container-id="ce103199-af03-4d37-bb35-24ad4e55653e", channel-max=32767, > idle-time-out=8000, offered-capabilities=@PN_SYMBOL[:"ANONYMOUS-RELAY"], > properties={:product="qpid-cpp", :version="0.35", :platform="Linux", > :host="localhost.localdomain"}] > {code} > The AMQP 1.0 spec does not make it clear that this is supported (e.g. see > diagram below) but in any case various components have shown difficulty with > it (such as PROTON-1135 just raised, and QPID-6639 which has yet to be > included in a release but permitted the above protocol trace log). > {code} > TCP Client TCP Server > = > AMQP%d3.1.0.0 -> > <- AMQP%d3.1.0.0 > : > : > > : > : > AMQP%d0.1.0.0 -> > (over SASL secured connection) > <- AMQP%d0.1.0.0 > open -> > <- open > {code} > Proton should by default disable sending pipelined OPEN frames for ANONYMOUS > logins, to aid compatibility with other components that don't expect/handle > such behaviour. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (PROTON-1135) [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default
[ https://issues.apache.org/jira/browse/PROTON-1135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217804#comment-15217804 ] Gordon Sim commented on PROTON-1135: My 2 cents: servers SHOULD handle the pipelined case (in the case of qpidd, I consider it a bug in the io handling rather than a concious choice, and I would guess the same is true of others). That said I do also think that proton is doing the wrong thing with sasl on the client and should wait for the server to offer mechanisms before choosing them. It MUST do this when a specific mechanism isn't chosen anyway (at least when integrated with cyrus), and even where it is I think it would provide clearer error information. Less of a concern (to me anyway) is pipelining the open before receiving a successful outcome. > [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default > - > > Key: PROTON-1135 > URL: https://issues.apache.org/jira/browse/PROTON-1135 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c >Affects Versions: 0.12.0 >Reporter: Ganesh Murthy > > Dispatch router (which uses Proton-c) currently sends pipelined SASL and OPEN > frames by default when connecting out to other peers using the ANONYMOUS > mech, as shown in the following trace - > {code} > [0x7f41f80079c0]: -> SASL > [0x7f41f80079c0]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, > initial-response=b"anonymous@localhost.localdomain"] > [0x7f41f80079c0]: -> AMQP > [0x7f41f80079c0]:0 -> @open(16) [container-id="Qpid.Dispatch.Router.A", > max-frame-size=65536, channel-max=32767, idle-time-out=8000, > offered-capabilities=:"ANONYMOUS-RELAY", > properties={:product="qpid-dispatch-router", :version="0.6.0"}] > [0x7f41f80079c0]: <- SASL > [0x7f41f80079c0]:0 <- @sasl-mechanisms(64) [sasl-server-mechanisms=:ANONYMOUS] > [0x7f41f80079c0]:0 <- @sasl-outcome(68) [code=0] > [0x7f41f80079c0]: <- AMQP > [0x7f41f80079c0]:0 <- @open(16) > [container-id="ce103199-af03-4d37-bb35-24ad4e55653e", channel-max=32767, > idle-time-out=8000, offered-capabilities=@PN_SYMBOL[:"ANONYMOUS-RELAY"], > properties={:product="qpid-cpp", :version="0.35", :platform="Linux", > :host="localhost.localdomain"}] > {code} > The AMQP 1.0 spec does not make it clear that this is supported (e.g. see > diagram below) but in any case various components have shown difficulty with > it (such as PROTON-1135 just raised, and QPID-6639 which has yet to be > included in a release but permitted the above protocol trace log). > {code} > TCP Client TCP Server > = > AMQP%d3.1.0.0 -> > <- AMQP%d3.1.0.0 > : > : > > : > : > AMQP%d0.1.0.0 -> > (over SASL secured connection) > <- AMQP%d0.1.0.0 > open -> > <- open > {code} > Proton should by default disable sending pipelined OPEN frames for ANONYMOUS > logins, to aid compatibility with other components that don't expect/handle > such behaviour. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (PROTON-1135) [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default
[ https://issues.apache.org/jira/browse/PROTON-1135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15214511#comment-15214511 ] Andrew Stitcher commented on PROTON-1135: - Fair enough - I certainly regard a comment from a spec author as more authoritative than my comments! In this case I'd say that we should stop pipelining SASL and AMQP protocol layers completely both client and server side. In this case my concern that we will be unable to test server pipelining no longer exists as we won't support that either. To be clear the complexities are implementing the mixed pipelining across layers - this change should not affect pipelining with only the AMQP layer present. And we should still be able to test that. > [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default > - > > Key: PROTON-1135 > URL: https://issues.apache.org/jira/browse/PROTON-1135 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c >Affects Versions: 0.12.0 >Reporter: Ganesh Murthy > > Dispatch router (which uses Proton-c) currently sends pipelined SASL and OPEN > frames by default when connecting out to other peers using the ANONYMOUS > mech, as shown in the following trace - > {code} > [0x7f41f80079c0]: -> SASL > [0x7f41f80079c0]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, > initial-response=b"anonymous@localhost.localdomain"] > [0x7f41f80079c0]: -> AMQP > [0x7f41f80079c0]:0 -> @open(16) [container-id="Qpid.Dispatch.Router.A", > max-frame-size=65536, channel-max=32767, idle-time-out=8000, > offered-capabilities=:"ANONYMOUS-RELAY", > properties={:product="qpid-dispatch-router", :version="0.6.0"}] > [0x7f41f80079c0]: <- SASL > [0x7f41f80079c0]:0 <- @sasl-mechanisms(64) [sasl-server-mechanisms=:ANONYMOUS] > [0x7f41f80079c0]:0 <- @sasl-outcome(68) [code=0] > [0x7f41f80079c0]: <- AMQP > [0x7f41f80079c0]:0 <- @open(16) > [container-id="ce103199-af03-4d37-bb35-24ad4e55653e", channel-max=32767, > idle-time-out=8000, offered-capabilities=@PN_SYMBOL[:"ANONYMOUS-RELAY"], > properties={:product="qpid-cpp", :version="0.35", :platform="Linux", > :host="localhost.localdomain"}] > {code} > The AMQP 1.0 spec does not make it clear that this is supported (e.g. see > diagram below) but in any case various components have shown difficulty with > it (such as PROTON-1135 just raised, and QPID-6639 which has yet to be > included in a release but permitted the above protocol trace log). > {code} > TCP Client TCP Server > = > AMQP%d3.1.0.0 -> > <- AMQP%d3.1.0.0 > : > : > > : > : > AMQP%d0.1.0.0 -> > (over SASL secured connection) > <- AMQP%d0.1.0.0 > open -> > <- open > {code} > Proton should by default disable sending pipelined OPEN frames for ANONYMOUS > logins, to aid compatibility with other components that don't expect/handle > such behaviour. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (PROTON-1135) [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default
[ https://issues.apache.org/jira/browse/PROTON-1135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15214500#comment-15214500 ] Rob Godfrey commented on PROTON-1135: - As a co-author of the AMQP specification, I don't think it can be taken as read that every implementation of a "server" MUST support pipelining of the SASL and AMQP layers together. The spec does lay out that the AMQP open sequence can be pipelined, but the SASL interaction is (in general) synchronous: the client must choose from the set of mechanisms offered. In certain circumstances the client (application) may have knowledge that allows it to operate in a pipelined manner (e.g. it knows that the server will accept the mechanism) or may be happy to fail in a reasonably unpleasant way if its assumptions are incorrect. However I think that it is unreasonable to expect that in general a peer will accept pipelining SASL and AMQP together, and empirical evidence shows that it is a poor assumption to make (as literally no other implementation has supported it out of the box). To my mind, in this scenario it would be more appropriate to omit the SASL layer entirely. > [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default > - > > Key: PROTON-1135 > URL: https://issues.apache.org/jira/browse/PROTON-1135 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c >Affects Versions: 0.12.0 >Reporter: Ganesh Murthy > > Dispatch router (which uses Proton-c) currently sends pipelined SASL and OPEN > frames by default when connecting out to other peers using the ANONYMOUS > mech, as shown in the following trace - > {code} > [0x7f41f80079c0]: -> SASL > [0x7f41f80079c0]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, > initial-response=b"anonymous@localhost.localdomain"] > [0x7f41f80079c0]: -> AMQP > [0x7f41f80079c0]:0 -> @open(16) [container-id="Qpid.Dispatch.Router.A", > max-frame-size=65536, channel-max=32767, idle-time-out=8000, > offered-capabilities=:"ANONYMOUS-RELAY", > properties={:product="qpid-dispatch-router", :version="0.6.0"}] > [0x7f41f80079c0]: <- SASL > [0x7f41f80079c0]:0 <- @sasl-mechanisms(64) [sasl-server-mechanisms=:ANONYMOUS] > [0x7f41f80079c0]:0 <- @sasl-outcome(68) [code=0] > [0x7f41f80079c0]: <- AMQP > [0x7f41f80079c0]:0 <- @open(16) > [container-id="ce103199-af03-4d37-bb35-24ad4e55653e", channel-max=32767, > idle-time-out=8000, offered-capabilities=@PN_SYMBOL[:"ANONYMOUS-RELAY"], > properties={:product="qpid-cpp", :version="0.35", :platform="Linux", > :host="localhost.localdomain"}] > {code} > The AMQP 1.0 spec does not make it clear that this is supported (e.g. see > diagram below) but in any case various components have shown difficulty with > it (such as PROTON-1135 just raised, and QPID-6639 which has yet to be > included in a release but permitted the above protocol trace log). > {code} > TCP Client TCP Server > = > AMQP%d3.1.0.0 -> > <- AMQP%d3.1.0.0 > : > : > > : > : > AMQP%d0.1.0.0 -> > (over SASL secured connection) > <- AMQP%d0.1.0.0 > open -> > <- open > {code} > Proton should by default disable sending pipelined OPEN frames for ANONYMOUS > logins, to aid compatibility with other components that don't expect/handle > such behaviour. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (PROTON-1135) [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default
[ https://issues.apache.org/jira/browse/PROTON-1135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15214365#comment-15214365 ] Andrew Stitcher commented on PROTON-1135: - As far as I can tell that the diagram in the description from the AMQP spec is not normative but merely descriptive. However there is a whole section in the standard "2.4.2 Pipelined Open" which strongly implies that pipelined open is a normative part of the protocol and so every conforming implementation MUST implement it. It is unfortunate the section 2.4.2 doesn't specifically discuss the implications of pipelined open to the SASL protocol interchange, but I think it is clear that where those frames can be pipelined they must be accepted too. I think that "open" in the sense of that paragraph implies the entire protocol up to and including sending the AMQP open frame. Client implementation are not required to pipeline, but server implementations must be able to cope with it as I read that paragraph. > [proton-c] dont pipeline SASL and OPEN frames for ANONYMOUS logins by default > - > > Key: PROTON-1135 > URL: https://issues.apache.org/jira/browse/PROTON-1135 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c >Affects Versions: 0.12.0 >Reporter: Ganesh Murthy > > Dispatch router (which uses Proton-c) currently sends pipelined SASL and OPEN > frames by default when connecting out to other peers using the ANONYMOUS > mech, as shown in the following trace - > {code} > [0x7f41f80079c0]: -> SASL > [0x7f41f80079c0]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, > initial-response=b"anonymous@localhost.localdomain"] > [0x7f41f80079c0]: -> AMQP > [0x7f41f80079c0]:0 -> @open(16) [container-id="Qpid.Dispatch.Router.A", > max-frame-size=65536, channel-max=32767, idle-time-out=8000, > offered-capabilities=:"ANONYMOUS-RELAY", > properties={:product="qpid-dispatch-router", :version="0.6.0"}] > [0x7f41f80079c0]: <- SASL > [0x7f41f80079c0]:0 <- @sasl-mechanisms(64) [sasl-server-mechanisms=:ANONYMOUS] > [0x7f41f80079c0]:0 <- @sasl-outcome(68) [code=0] > [0x7f41f80079c0]: <- AMQP > [0x7f41f80079c0]:0 <- @open(16) > [container-id="ce103199-af03-4d37-bb35-24ad4e55653e", channel-max=32767, > idle-time-out=8000, offered-capabilities=@PN_SYMBOL[:"ANONYMOUS-RELAY"], > properties={:product="qpid-cpp", :version="0.35", :platform="Linux", > :host="localhost.localdomain"}] > {code} > The AMQP 1.0 spec does not make it clear that this is supported (e.g. see > diagram below) but in any case various components have shown difficulty with > it (such as PROTON-1135 just raised, and QPID-6639 which has yet to be > included in a release but permitted the above protocol trace log). > {code} > TCP Client TCP Server > = > AMQP%d3.1.0.0 -> > <- AMQP%d3.1.0.0 > : > : > > : > : > AMQP%d0.1.0.0 -> > (over SASL secured connection) > <- AMQP%d0.1.0.0 > open -> > <- open > {code} > Proton should by default disable sending pipelined OPEN frames for ANONYMOUS > logins, to aid compatibility with other components that don't expect/handle > such behaviour. -- This message was sent by Atlassian JIRA (v6.3.4#6332)