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

NioSocketConnector.newHandle throws the wrong exception

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.9
    • Fix Version/s: 2.0.10
    • Component/s: Core
    • Labels:
      None

      Description

      NioSocketConnector.newHandle creates an IOException which is then not used. I assume this meant to be thrown instead of the original exception. That is "throw ioe;" should be replaced with "throw e"; (or simply rethrow the original exception, so that the current behaviour does not change).

        Activity

        Hide
        elecharny Emmanuel Lecharny added a comment -

        Fixed with commit d56e27f6b31c7b776f401f942d69bad29fbebac4

        Show
        elecharny Emmanuel Lecharny added a comment - Fixed with commit d56e27f6b31c7b776f401f942d69bad29fbebac4
        Hide
        elecharny Emmanuel Lecharny added a comment -

        Thanks for the report ! The exception is correctly send in the three other classes ( for Datagram acceptor and connector and for the SicketAcceptor) butsadly not for the SockectConnector.

        I just pushed a fix for that.

        Show
        elecharny Emmanuel Lecharny added a comment - Thanks for the report ! The exception is correctly send in the three other classes ( for Datagram acceptor and connector and for the SicketAcceptor) butsadly not for the SockectConnector. I just pushed a fix for that.

          People

          • Assignee:
            Unassigned
            Reporter:
            kelemen Attila Kelemen
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development