[
https://issues.apache.org/jira/browse/DIRMINA-524?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Mike Heath resolved DIRMINA-524.
--------------------------------
Resolution: Fixed
Fixed, thanks Geoff for the patch.
> ProtocolCodecFilter should pass FileRegion to nextFilter in addition to
> IoBuffer
> --------------------------------------------------------------------------------
>
> Key: DIRMINA-524
> URL: https://issues.apache.org/jira/browse/DIRMINA-524
> Project: MINA
> Issue Type: Bug
> Components: Filter
> Environment: jdk1.6.0_03 Linux x64
> Reporter: Geoff Cadien
> Fix For: 2.0.0-M1
>
>
> ProtocolCodecFilter.filterWrite calls nextFilter.filterWrite directly without
> invoking the ProtocolEncoder if the WriteRequest message is an IoBuffer. It
> should do the same if the message is a FileRegion.
> Here is a quick patch:
> Index: ProtocolCodecFilter.java
> ===================================================================
> --- ProtocolCodecFilter.java (revision 617648)
> +++ ProtocolCodecFilter.java (working copy)
> @@ -25,6 +25,7 @@
> import org.apache.mina.common.AttributeKey;
> import org.apache.mina.common.DefaultWriteFuture;
> import org.apache.mina.common.DefaultWriteRequest;
> +import org.apache.mina.common.FileRegion;
> import org.apache.mina.common.IoBuffer;
> import org.apache.mina.common.IoFilter;
> import org.apache.mina.common.IoFilterAdapter;
> @@ -222,7 +223,7 @@
> public void filterWrite(NextFilter nextFilter, IoSession session,
> WriteRequest writeRequest) throws Exception {
> Object message = writeRequest.getMessage();
> - if (message instanceof IoBuffer) {
> + if (message instanceof IoBuffer || message instanceof FileRegion) {
> nextFilter.filterWrite(session, writeRequest);
> return;
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
