[ https://issues.apache.org/jira/browse/DISPATCH-803?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120385#comment-16120385 ]
ASF GitHub Bot commented on DISPATCH-803: ----------------------------------------- Github user ted-ross commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/185#discussion_r132261405 --- Diff: src/router_core/transfer.c --- @@ -634,6 +634,23 @@ static void qdr_link_forward_CT(qdr_core_t *core, qdr_link_t *link, qdr_delivery addr->deliveries_ingress++; link->total_deliveries++; } + // + // There is no address that we can send this delivery to, which means the addr was not found in our hastable. This + // can be because there were no receivers or because the address was not defined in the config file. + // If the treatment for such addresses is set to be forbidden, we send back a rejected disposition and detach the link + // + else if (core->qd->treatment == QD_TREATMENT_LINK_FORBIDDEN) { + dlv->disposition = PN_REJECTED; + dlv->error = qdr_error("qd:forbidden", "Sending deliveries to this address is forbidden"); --- End diff -- You should be using one of the AMQP-specified error conditions that is in the amqp.[ch] files. It seems that the appropriate error condition is "amqp:not-allowed", not "forbidden" (which is an error code only for the management protocol). > refuse attach to undefined addresses > ------------------------------------ > > Key: DISPATCH-803 > URL: https://issues.apache.org/jira/browse/DISPATCH-803 > Project: Qpid Dispatch > Issue Type: Improvement > Reporter: Gordon Sim > Assignee: Ganesh Murthy > Fix For: 1.0.0 > > > At present, if you attach to an address in the router whose semantics have > not been specifically defined, you get balanced message routing semantics. > It would be useful to be able to configure the router such that it would > refuse links whose source/target was not explicitly defined. E.g. by being > able to configure the default semantics to be of type 'invalid' (or anything > similar). (Being able to explicitly blacklist certain addresses might also be > nice, but is a more exotic use case I think). > Messages sent through an anonymous link to these 'invalid' addresses would be > rejected. > -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org