Qpid
  1. Qpid
  2. QPID-4562

Message#reply_to= should auto-wrap the address argument in an Address if the argument is a String

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21
    • Component/s: Ruby Client
    • Labels:
      None

      Description

      It would be nice to be able to write

      message.reply_to = "exchange/subject;

      {...}

      "

      Message#reply_to= currently requires its argument to be an instance of the Address class. It would be nice if it would check the class, and if it's a String, wrap it in an Address automatically.

        Activity

        Andy Goldstein created issue -
        Andy Goldstein made changes -
        Field Original Value New Value
        Summary qpid_messaging gem doesn't support message.reply_to = 'name/subject; {node:{...}}' Message#reply_to= should auto-wrap the address argument in an Address if the argument is a String
        Description The qpid_messaging gem currently requires the argument passed to message.reply_to= to be an Address. The other languages (c++, python) allow you to pass a string to reply_to=. When trying to construct an address using Qpid::Messaging::Address.new('qmf.default.topic', 'direct.xyz', :node => {:type => :topic}), Ruby segfaults because of a problem converting a string to a map. It would be nice if it were possible to simply pass a string to Message#reply_to=. It would be nice to be able to write

        message.reply_to = "exchange/subject; {...}"

        Message#reply_to= currently requires its argument to be an instance of the Address class. It would be nice if it would check the class, and if it's a String, wrap it in an Address automatically.
        Darryl L. Pierce made changes -
        Assignee Darryl L. Pierce [ mcpierce ]
        Darryl L. Pierce made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.21 [ 12323549 ]
        Resolution Fixed [ 1 ]
        Justin Ross made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Darryl L. Pierce
            Reporter:
            Andy Goldstein
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development