Bug 1087852 - Org.Apache.Qpid.Messaging.Message.SetContentObject(object) method cannot set Dictionary<> or Collection<> types as message content
Summary: Org.Apache.Qpid.Messaging.Message.SetContentObject(object) method cannot set ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: qpid-sdk
Version: Development
Hardware: Unspecified
OS: Windows
medium
high
Target Milestone: 3.1
: ---
Assignee: Chuck Rolke
QA Contact: Petra Svobodová
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-15 12:58 UTC by Petra Svobodová
Modified: 2015-04-14 13:47 UTC (History)
4 users (show)

Fixed In Version: qpid-cpp-0.30-2
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-04-14 13:47:55 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Modified C# helloworld example to reproduce this issue (2.70 KB, text/x-csharp)
2014-04-15 12:58 UTC, Petra Svobodová
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Apache JIRA QPID-5964 0 None None None Never
Red Hat Product Errata RHEA-2015:0805 0 normal SHIPPED_LIVE Red Hat Enterprise MRG Messaging 3.1 Release 2015-04-14 17:45:54 UTC

Description Petra Svobodová 2014-04-15 12:58:25 UTC
Created attachment 886482 [details]
Modified C# helloworld example to reproduce this issue

Description of problem:

The SetContentObject(object) method is able to fill in the message content with given string argument, however if the argument is Dictionary<> or Collection<> type, the method raises an exception:

"C:\qpid_VS2010\bin\Release>csharp.example.helloworld.exe 10.34.37.200
Exception System.NotImplementedException: The method or operation is not impleme
nted.
   at Org.Apache.Qpid.Messaging.TypeTranslator.ManagedToNativeObject(Object mana
gedValue, Variant* qpidVariant) in q:\qpid\cpp\bindings\qpid\dotnet\src\typetran
slator.cpp:line 247
   at Org.Apache.Qpid.Messaging.Message.SetContentObject(Object managedObject) i
n q:\qpid\cpp\bindings\qpid\dotnet\src\message.cpp:line 418
   at Org.Apache.Qpid.Messaging.Program.Main(String[] args) in c:\qpid_VS2010\do
tnet_examples\examples\csharp.example.helloworld\csharp.example.helloworld.cs:li
ne 49."

Version-Release number of selected component (if applicable):
qpid-cpp-win-3.22.37.1-1

How reproducible:
100%

Steps to Reproduce:
1. Unpack the qpid-cpp-win package.
2. Compile the attached reproducer.
3. Start qpid broker and try to run the reproducer: csharp.example.helloworld.exe <broker_hostname>

Actual results:
SetContentObject() method raises the exception (see upper, please).

Expected results:
SetContentObject() method should fill in the message content from the given object.

Comment 1 Justin Ross 2014-05-02 10:42:59 UTC
Chuck, is there any workaround for this?

Comment 2 Chuck Rolke 2014-05-02 20:40:44 UTC
SetContentObject and GetContentObject fail to return the structured map and list object types.

A workaround for this issue is to detect the message content type and then fetch the message into the proper container.

  static int Main(string[] args)
  {
    Dictionary<string, object> mapContent = new Dictionary<string, object>();
    Message mapMsg = new Message(mapContent);
    Console.WriteLine("Map  ContentType = {0}", mapMsg.ContentType);

    Collection<Object> colContent = new Collection<object>();
    Message colMsg = new Message(colContent);
    Console.WriteLine("List ContentType = {0}", colMsg.ContentType);

    // Create message of some amqp type then discover that type
    // and retrieve the message content.
    Message someMsg = new Message("abc");       // string
    //Message someMsg = new Message(mapContent);   // map
    //Message someMsg = new Message(colContent);   // list

    if ("amqp/map" == someMsg.ContentType)
    {
      Dictionary<string, object> mapContentRcvd = 
        new Dictionary<string, object>();
      someMsg.GetContent(mapContentRcvd);
    }
    else if ("amqp/list" == someMsg.ContentType)
    {
      Collection<Object> colContentRcvd = new Collection<object>();
      someMsg.GetContent(colContentRcvd);
    }
    else
    {
      string stringContentRcvd = "";
      stringContentRcvd = someMsg.GetContent();
    }
    return 0;
  }

Comment 3 Justin Ross 2014-05-05 11:38:58 UTC
What does Message.GetContent(someArg) do?  There's no equivalent in the C++ api, and it's not documented in the .NET api.

Is it an out param?  If so, what is motivating us to have such things in .NET?  They're not idiomatic.

The workaround provided is mostly about extracting map and collection data, but the reported problem is *setting* it.  If the workaround is as simple as "use the constructor", that's what we should say.

Comment 5 Chuck Rolke 2014-06-23 20:59:11 UTC
This is still an issue:

                Collection<Object> l = new Collection<Object>();
                l.Add("hello");
                l.Add("world");
                msg.SetContentObject(l);

The underlying type marshal code does not properly convert Collection (amqp/list) and Dictionary (amqp/map) at all. The SetContentObject code converts primitive objects like int and string only.

Comment 8 Chuck Rolke 2014-08-05 20:01:12 UTC
Commited upstream at r1615992

Comment 10 Petra Svobodová 2014-12-11 13:36:26 UTC
Retested on qpid-cpp-win-3.30.4.1-1:
Now the SetContentObject method is able to set objects of System.Collections.Generic.Dictionary<T> or System.String types as message content.
When I tried to set System.Collections.ObjectModel.Collection<T> type object as the message content the issue still persists, SetContentObject() raises the Exception.

Comment 11 Petra Svobodová 2015-02-12 14:44:43 UTC
Now the SetContentObject(object) method can accept System.Collections.ObjectModel.Collection<object> and System.Collections.Generic.Dictionary<string, object> types to fill in the message content.

Verified on qpid-cpp-win-3.30.6.1-1 on Windows 7-x64, Windows 8-x64, Windows Server 2008-x64 and R2 and Windows Server 2012 R2.

--> VERIFIED

Thank you for the patch!

Comment 14 errata-xmlrpc 2015-04-14 13:47:55 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHEA-2015-0805.html


Note You need to log in before you can comment on or make changes to this bug.