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

Immediate session close with a SSL filter

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Not A Problem
    • Affects Version/s: 2.0.7
    • Fix Version/s: 2.0.8
    • Component/s: Filter
    • Labels:
      None

      Description

      In the filterClose call in the SSLFilter , we try to write a last message in the queue and we add a listener to only continue to the next filter once that message is written. So when the client is already backed up like crazy and the os buffer is full, doing this is pretty much useless as you have to wait for the entire queue of backed up messages to be consumed and the session just sit there.

      So in that case, i would think that the right behavior would be to clear the queue before to try to write that last ssl close message and even have a timer on the message and if it can not be written in x sec, screw the clean ssl termination just closed the socket.

        Activity

        Hide
        elecharny Emmanuel Lecharny added a comment -

        The pending messages are just pushed into the IoProcessor queue, nothing more. The session will not get blocked.

        Show
        elecharny Emmanuel Lecharny added a comment - The pending messages are just pushed into the IoProcessor queue, nothing more. The session will not get blocked.
        Hide
        elecharny Emmanuel Lecharny added a comment -

        Also note that just because the server has close the connection does not mean the enqueued messages should be discarded.

        Show
        elecharny Emmanuel Lecharny added a comment - Also note that just because the server has close the connection does not mean the enqueued messages should be discarded.

          People

          • Assignee:
            Unassigned
            Reporter:
            rrossier Romain Rossier
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development