[CAMEL-2237] Improper ordering of MINA filters

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.2.0
• Component/s: camel-mina
• Patch Info: Patch Available
• Assignee: Martin Krasser
• Reporter: Dmytro Rud
• Created: 30/Nov/09 09:46
• Updated: 03/Nov/10 05:12
• Resolved: 01/Dec/09 14:31

Description

When we want to use SSL on the consumer side, MINA's SSLFilter must be inserted as the very first filter into the chain (or, at least, as the second one after an ExecutorFilter), but it is currently not possible, because MinaComponent#createSocketEndpoint() always inserts a ProtocolCodecFilter at the very beginning by calling configureCodecFactory().

A proposed workaround is to introduce an additional URL parameter noDefaultCodec which prohibits the installation of the default protocol codec filter. The protocol codec must be then configured manually, e.g. using Spring:

<bean id="sslFilter" class="org.apache.mina.filter.SSLFilter">
<constructor-arg>
<bean class="javax.net.ssl.SSLContext" factory-method="getDefault" />
</constructor-arg>
</bean>

<bean id="codecFilter" class="org.apache.mina.filter.codec.ProtocolCodecFilter">
<constructor-arg ref="hl7codec" />
</bean>

<bean id="hl7codec" class="org.apache.camel.component.hl7.HL7MLLPCodec" />

<bean id="minaFilters" class="java.util.ArrayList">
<constructor-arg>
<list value-type="org.apache.mina.common.IoFilter">
<ref bean="sslFilter" />
<ref bean="codecFilter" />
</list>
</constructor-arg>
</bean>

After that, the consumer endpoint URL can look like
from("mina:tcp://0.0.0.0:8888?sync=true&lazySessionCreation=true&noDefaultCodec=true&filters=#minaFilters").

I am not sure whether this approach is optimal, therefore I call it "workaround" and not "solution". The corresponding patch is attached.