[ https://issues.apache.org/jira/browse/QPID-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15222005#comment-15222005 ]
Alex Rudyy commented on QPID-7159: ---------------------------------- Hi Jakub, Your second patch adding a connection parameter "attachUserId" looks good to me. However, I would like to suggest renaming of connection parameter "attachUserId" into "populateJMSXUserID" in order to use a similar name to a connection parameter used in new JMS client for the same purpose. (See [QPIDJMS-163|https://issues.apache.org/jira/browse/QPIDJMS-163]). New JMS client parameter name is "jms.populateJMSXUserID". It has a prefix "jms." which I would suggested to omit as at the moment none of existing connection parameter in legacy JMS client has prefix. > [Java Client] Disabling user ids in AMQP messages > ------------------------------------------------- > > Key: QPID-7159 > URL: https://issues.apache.org/jira/browse/QPID-7159 > Project: Qpid > Issue Type: New Feature > Components: Java Client > Reporter: Jakub Scholz > Fix For: qpid-java-6.0.2, qpid-java-6.1 > > Attachments: QPID-7159-conn-option.patch, QPID-7159.patch > > > The AMQP 0-10 JMS client seems to always attach the user-id message property > to the AMQP message. > In some cases, I don't think this is desired. For example when I send > messages to my customers, I do not really want them to know which other users > exist on the broker and are used to send messages. > It would be great to have a possibility to disable the user-id when needed. > The attached patch disables the user id in the BasicMessageProducer based on > a system property qpid.attach_user_id (true/false). Does that look like a > reasonable solution? If yes, I can try to add some tests. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org