New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How to support SOAP 1.2 over JMS #268

Closed
uportalis opened this Issue Jul 21, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@uportalis

uportalis commented Jul 21, 2017

Hi

I'm using Citrus Framework 2.7.2.
I can't find how to support SOAP 1.2 over JMS.

Here is my context configuration:

	<citrus-jms:sync-endpoint id="my_soap_q" 
		destination-name="myqueuename"
		message-converter="soapMessageConverter"
		timeout="8000" connection-factory="myConnectionFactory" />

	<citrus-jms:sync-endpoint id="my_soap12_q" 
		destination-name="myqueuename"
		message-converter="soap12MessageConverter"
		timeout="8000" connection-factory="myConnectionFactory" />

	<bean id="messageFactory" class="org.springframework.ws.soap.saaj.SaajSoapMessageFactory">
		<property name="soapVersion">
			<util:constant static-field="org.springframework.ws.soap.SoapVersion.SOAP_11" />
		</property>
	</bean>

	<bean id="soapMessageConverter" class="com.consol.citrus.jms.message.SoapJmsMessageConverter">
		<property name="soapMessageFactory" ref="soapMessageFactory" />
	</bean>

	<bean id="soap12MessageFactory" class="org.springframework.ws.soap.saaj.SaajSoapMessageFactory">
		<property name="soapVersion">
			<util:constant static-field="org.springframework.ws.soap.SoapVersion.SOAP_12" />
		</property>
	</bean>

	<bean id="soap12MessageConverter" class="com.consol.citrus.jms.message.SoapJmsMessageConverter">
		<!-- TODO: set its soapMessageFactory to the bean "soap12MessageFactory" -->
	</bean>

I could not find how to get the bean "soap12MessageFactory" autowired into the bean "soap12MessageConverter" (see the "TODO" in my configuration above).

Any idea?

rgds,
Louis

@christophd

This comment has been minimized.

Member

christophd commented Sep 26, 2017

This is a bug, we need to add setter for message factory in this class. Also we need to make sure that autowiring is not broken by two or more message factories present in application context. Stay tuned for a fix please and thanks for pointing to this issue!

@christophd christophd added this to the v2.7.3 milestone Sep 26, 2017

@christophd christophd added IN PROGRESS and removed READY labels Oct 5, 2017

@christophd christophd self-assigned this Oct 5, 2017

@christophd christophd closed this in 19c18f8 Oct 5, 2017

@christophd christophd removed the IN PROGRESS label Oct 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment