Uploaded image for project: 'MINA'
  1. MINA
  2. DIRMINA-1033

Expose METADATA parameter for NioDatagramSession

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.0.13
    • Fix Version/s: None
    • Component/s: Transport
    • Labels:
    • Environment:
      Centos 7, Windows 7
    • Flags:
      Patch

      Description

      The CumulativeProtocolHandler doesn't accumulate for UDP transport.
      It's because METADATA for NioDatagramSession is created as

      static final TransportMetadata METADATA = new DefaultTransportMetadata("nio", "datagram", true, false, InetSocketAddress.class, DatagramSessionConfig.class, IoBuffer.class);

      Where 4th parameter is fragmentation parameter. Thus it thinks there are no fragmentation for UDP transport.

      NioDatagramSession METADATA should be created as:
      static final TransportMetadata METADATA = new DefaultTransportMetadata("nio", "datagram", true, true,
      InetSocketAddress.class, DatagramSessionConfig.class, IoBuffer.class);

      Where 4th parameter is true, marking fragmentation is true for UDP.

      Or the METADATA parameter can be exposed to developer.

      This is link to the mailing list discussion.
      http://apache-mina.10907.n7.nabble.com/CumulativeProtocolDecoder-and-UDP-td18927.html

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              tuncatunc Tunca Tunc
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:

                Time Tracking

                Estimated:
                Original Estimate - 1h
                1h
                Remaining:
                Remaining Estimate - 1h
                1h
                Logged:
                Time Spent - Not Specified
                Not Specified