Just to let you know, it works now. I had to do a little juggling with the
JAXBMarshaller so it is able to handle the types I'm passing it, but it
looks better now. I'll figure out how I can patch to into a better form.
On 9/20/07, Guillaume Nodet <[EMAIL PROTECTED]> wrote:
>
> I was a bit puzzled when I first read your mail because this would
> have implied that the NotificationBroker has never worked :-( But now
> I've understood the problem.
> If you give a JBIContainer or ComponentContext when constructing the
> NotifiationBroker object, it creates a client with a JAXBMarshaler
> which overrides the very method that fails. Unfortunately, when
> giving a ServiceMixClient, the marshaler is not changed and the
> DefaultMarshaler is used.
> I guess it should be possible to set a jaxb marshaler on the client
> before constructing the NotificationBroker to workaround your
> problem... but I guess it would be better if the the marshaler is
> automatically set to a JAXB one instead.
>
> I suppose the same thing may happen with the other class in the wsn
> client api...
> Wanna raise a jira and provide a patch ?
>
> On 9/21/07, Ryan Moquin <[EMAIL PROTECTED]> wrote:
> > Ok, I think this is a bug in some form (that the very least and
> exception or
> > message should be logged). It appears that servicemix doesn't like my
> > object is silently tosses it into the trash:
> >
> > This method (in NotificationBroker) gets executed with my object and a
> > org.oasis_open.docs.wsn.b_2.Notify is constructed:
> >
> > public void notify(String topic, Object msg) throws JBIException {
> > Notify notify = new Notify();
> > NotificationMessageHolderType holder = new
> > NotificationMessageHolderType();
> > if (topic != null) {
> > TopicExpressionType topicExp = new TopicExpressionType();
> > topicExp.getContent().add(topic);
> > holder.setTopic(topicExp);
> > }
> > holder.setMessage(new NotificationMessageHolderType.Message());
> > holder.getMessage().setAny(msg);
> > notify.getNotificationMessage().add(holder);
> > send(notify);
> > }
> >
> > Eventually this method gets executed (in DefaultMarshaller) with the
> Notify
> > object that was created as the parameter in body, but since the Notify
> > object isn't an instance of any of the 3 things it's looking for, it
> returns
> > null (actually the message variable isn't even used in this method):
> >
> > protected Source asContent(NormalizedMessage message, Object body) {
> > if (body instanceof Source) {
> > return (Source) body;
> > } else if (body instanceof String) {
> > // lets assume String is the XML to send
> > return new StringSource((String) body);
> > } else if (body instanceof Node) {
> > return new DOMSource((Node) body);
> > }
> > return null;
> > }
> >
> > This null then is put into the message variable as the content (line 56
> on
> > DefaultMarshaller), hence my object gets tossed:
> >
> > Source content = asContent(message, body);
> > message.setContent(content);
> >
> > This was all called by DefaultServicemixClient, which returns back to
> the
> > following method, with the in variable now populated with null as the
> > content:
> >
> > protected void populateMessage(MessageExchange exchange, Map
> > exchangeProperties,
> > Map inMessageProperties, Object
> content)
> > throws MessagingException {
> > NormalizedMessage in = exchange.getMessage("in");
> > populateExchangeProperties(exchange, exchangeProperties);
> > populateMessageProperties(in, inMessageProperties);
> > getMarshaler().marshal(exchange, in, content);
> > }
> >
> > I'll jira this since an action item of some kind should definitely be
> done,
> > I'd go ahead and patch it but I'm not completely sure what kind of class
> > it's expecting in the asContent method yet.
> >
> > On 9/20/07, Ryan Moquin <[EMAIL PROTECTED]> wrote:
> > >
> > > I did debug my app and it is submitting an object that is not null,
> which
> > > is why this is puzzling. Initially I thought maybe I was trying to
> through
> > > a null object in there. I'll do some poking around and see if I can
> find
> > > where it might be going haywire.
> > >
> > > On 9/20/07, Guillaume Nodet <[EMAIL PROTECTED]> wrote:
> > > >
> > > > The client does not send anything for an unknown reason.
> > > > You can see that because the in message has a null content.
> > > > This is what causes the exception.
> > > > I suggest to remote debug the app to see what happens...
> > > > Just set the environment variable SERVICEMIX_DEBUG to TRUE, launch
> > > > servicemix and debug from eclipse or your favorite ide...
> > > >
> > > > On 9/20/07, Ryan Moquin <[EMAIL PROTECTED] > wrote:
> > > > > I don't get much more, but here is what I see now:
> > > > >
> > > > > DEBUG - WSNComponent - Received exchange:
> status:
> > > > Active,
> > > > > role: provider
> > > > > DEBUG - WSNComponent - Retrieved correlation id:
> > > > null
> > > > > ERROR - WSNComponent - Error processing exchange
> > > > InOnly[
> > > > > id: ID:10.40.16.154-1152419544a-28:1
> > > > > status: Active
> > > > > role: provider
> > > > > endpoint: Broker
> > > > > in: null
> > > > > ]
> > > > > java.lang.IllegalArgumentException: source parameter must not be
> null
> > > > > at
> javax.xml.bind.helpers.AbstractUnmarshallerImpl.unmarshal(
> > > > > AbstractUnmarshallerImpl.java:98)
> > > > > at org.apache.servicemix.wsn.component.WSNEndpoint.process
> (
> > > > > WSNEndpoint.java:128)
> > > > > at
> org.apache.servicemix.common.AsyncBaseLifeCycle.doProcess(
> > > > > AsyncBaseLifeCycle.java:538)
> > > > > at
> > > > org.apache.servicemix.common.AsyncBaseLifeCycle.processExchange(
> > > > > AsyncBaseLifeCycle.java:490)
> > > > > at
> > > > org.apache.servicemix.common.BaseLifeCycle.onMessageExchange(
> > > > > BaseLifeCycle.java:46)
> > > > > at
> > > > >
> org.apache.servicemix.jbi.messaging.DeliveryChannelImpl.processInBound
> > > > (
> > > > > DeliveryChannelImpl.java:610)
> > > > > at
> org.apache.servicemix.jbi.nmr.flow.AbstractFlow.doRouting(
> > > > > AbstractFlow.java :170)
> > > > > at
> org.apache.servicemix.jbi.nmr.flow.seda.SedaFlow.doRouting(
> > > > > SedaFlow.java:167)
> > > > > at org.apache.servicemix.jbi.nmr.flow.seda.SedaQueue$1.run
> (
> > > > > SedaQueue.java:134)
> > > > > at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(
> > > > > ThreadPoolExecutor.java:650)
> > > > > at java.util.concurrent.ThreadPoolExecutor$Worker.run(
> > > > > ThreadPoolExecutor.java:675)
> > > > > at java.lang.Thread.run (Thread.java:595)
> > > > > DEBUG - DeliveryChannelImpl - Send ID:
> > > > 10.40.16.154-1152419544a-28:1
> > > > > in DeliveryChannel{servicemix-wsn2005}
> > > > > DEBUG - SedaFlow - Called Flow send
> > > > > DEBUG - SedaQueue -
> > > > > [EMAIL PROTECTED]
> > > > > exchange: I
> > > > > nOnly[
> > > > > id: ID:10.40.16.154-1152419544a-28:1
> > > > > status: Error
> > > > > role: consumer
> > > > > endpoint: Broker
> > > > > in: null
> > > > > error: java.lang.IllegalArgumentException: source parameter must
> not
> > > > be
> > > > > null
> > > > > ]
> > > > > DEBUG - DeliveryChannelImpl - Notifying exchange ID:
> > > > > 10.40.16.154-1152419544a-28:1(e99889) in DeliveryChannel{I
> > > > > D:10.40.16.154-1152419544a-0:1} from
> processInboundSynchronousExchange
> > > > > DEBUG - DeliveryChannelImpl - Notified: ID:
> > > > > 10.40.16.154-1152419544a-28 :1(e99889) in DeliveryChannel{ID:
> 10.40.1
> > > > > 6.154-1152419544a-0:1} from sendSync
> > > > >
> > > > > On 9/20/07, Guillaume Nodet <[EMAIL PROTECTED]> wrote:
> > > > > >
> > > > > > It seems that nothing is received by then WS-Notification
> component.
> > > >
> > > > > > Can you raise the log level to DEBUG and see what is displayed ?
> > > > > >
> > > > > > On 9/20/07, Ryan Moquin <[EMAIL PROTECTED]> wrote:
> > > > > > > It's actually the JAXB objects I generated with CXF for my
> > > > webservices.
> > > > > > > Those should be JAXB2 elements. They are the elements I
> return
> > > > from my
> > > > > > > webservice to a connected client.
> > > > > > >
> > > > > > > On 9/20/07, Guillaume Nodet < [EMAIL PROTECTED]> wrote:
> > > > > > > >
> > > > > > > > What kind of object do you send ? IIRC it has to be a DOM
> > > > element or a
> > > > > > > > JAXB2 pojo so that JAXB2 can marshal it to xml.
> > > > > > > >
> > > > > > > > On 9/20/07, Ryan Moquin <[EMAIL PROTECTED]> wrote:
> > > > > > > > > I'm having some trouble when attempting to publish a
> > > > > > notification. When
> > > > > > > > I
> > > > > > > > > do, I get this error:
> > > > > > > > >
> > > > > > > > > ERROR - WSNComponent - Error processing
> > > > exchange
> > > > > > > > InOnly[
> > > > > > > > > id: ID:192.168.1.2-11520deb240-28:0
> > > > > > > > > status: Active
> > > > > > > > > role: provider
> > > > > > > > > endpoint: Broker
> > > > > > > > > in: null
> > > > > > > > > ]
> > > > > > > > > java.lang.IllegalArgumentException: source parameter must
> not
> > > > be
> > > > > > null
> > > > > > > > > at
> > > > javax.xml.bind.helpers.AbstractUnmarshallerImpl.unmarshal
> > > > > > (
> > > > > > > > > AbstractUnmarshallerImpl.java:98)
> > > > > > > > > at
> > > > org.apache.servicemix.wsn.component.WSNEndpoint.process(
> > > > > > > > > WSNEndpoint.java:128)
> > > > > > > > > at
> > > > org.apache.servicemix.common.AsyncBaseLifeCycle.doProcess
> > > > > > (
> > > > > > > > > AsyncBaseLifeCycle.java:538)
> > > > > > > > > at
> > > > > > > >
> org.apache.servicemix.common.AsyncBaseLifeCycle.processExchange(
> > > > > > > > > AsyncBaseLifeCycle.java:490)
> > > > > > > > >
> > > > > > > > > My code that causes this is:
> > > > > > > > >
> > > > > > > > > private void sendNotification(Object notification) {
> > > > > > > > > try {
> > > > > > > > > if(wsnBroker == null)
> > > > > > > > > createWsnBroker();
> > > > > > > > > wsnBroker.notify("notificationTopic", notification);
> > > > > > > > > } catch (Exception e) {
> > > > > > > > > e.printStackTrace();
> > > > > > > > > }
> > > > > > > > > }
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > I've looked and my notification object isn't null, is
> there
> > > > > > something
> > > > > > > > called
> > > > > > > > > source somewhere that I need to set?
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > > Cheers,
> > > > > > > > Guillaume Nodet
> > > > > > > > ------------------------
> > > > > > > > Blog: http://gnodet.blogspot.com/
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Cheers,
> > > > > > Guillaume Nodet
> > > > > > ------------------------
> > > > > > Blog: http://gnodet.blogspot.com/
> > > > > >
> > > > >
> > > >
> > > >
> > > > --
> > > > Cheers,
> > > > Guillaume Nodet
> > > > ------------------------
> > > > Blog: http://gnodet.blogspot.com/
> > > >
> > >
> > >
> >
>
>
> --
> Cheers,
> Guillaume Nodet
> ------------------------
> Blog: http://gnodet.blogspot.com/
>
