UDP server respawn after timeout












0














I'm writing an Android application (API 24) that should receive telemetry messages trough an UDP connection. It works fine if the (wireless) connection is in good state, as long as it degrades the UDP receiver stops working and I'm not able to respawn even when the connection it's again in good state.



This's my code:



private class UDPReceiver extends Thread {
private boolean keepRunning = true;
private int UDP_SERVER_PORT;
public Queue<String> messagesQueue = new LinkedList<>();

public void run() {
byte buffer = new byte[MAX_UDP_DATAGRAM_LEN];

DatagramPacket packet = new DatagramPacket(buffer, buffer.length);
DatagramSocket socket = null;

do {
try {
socket = new DatagramSocket(UDP_SERVER_PORT);
socket.setReuseAddress(true);

while (keepRunning) {
socket.receive(packet);
messagesQueue.add(new String(buffer, 0, packet.getLength()));
runOnUiThread(updateTelemetry);
}
} catch (Throwable e) {
Log.e(this.getClass().getName(), e.getMessage());
e.printStackTrace();
}

try {
Thread.sleep(UDP_SERVER_RESPAWN);
} catch (InterruptedException e) {
e.printStackTrace();
}
} while (keepRunning);

if (socket != null) {
socket.close();
}
}

private Runnable updateTelemetry = new Runnable() {
public void run() {
if (gcsUDPReceiver == null) return;

final String message = gcsUDPReceiver.messagesQueue.poll();

if (message == null) return;

telemetryTask(message);
}
};


I thought that the outer do...while loop in UDPReceiver should fix the problem respawning the dead connection but this's not working as I expected, moreover setReuseAddress(true) directive seems to be a source of issues instead a partial fix.



I'm probably doing some other error but I'm not able to catch it.










share|improve this question





























    0














    I'm writing an Android application (API 24) that should receive telemetry messages trough an UDP connection. It works fine if the (wireless) connection is in good state, as long as it degrades the UDP receiver stops working and I'm not able to respawn even when the connection it's again in good state.



    This's my code:



    private class UDPReceiver extends Thread {
    private boolean keepRunning = true;
    private int UDP_SERVER_PORT;
    public Queue<String> messagesQueue = new LinkedList<>();

    public void run() {
    byte buffer = new byte[MAX_UDP_DATAGRAM_LEN];

    DatagramPacket packet = new DatagramPacket(buffer, buffer.length);
    DatagramSocket socket = null;

    do {
    try {
    socket = new DatagramSocket(UDP_SERVER_PORT);
    socket.setReuseAddress(true);

    while (keepRunning) {
    socket.receive(packet);
    messagesQueue.add(new String(buffer, 0, packet.getLength()));
    runOnUiThread(updateTelemetry);
    }
    } catch (Throwable e) {
    Log.e(this.getClass().getName(), e.getMessage());
    e.printStackTrace();
    }

    try {
    Thread.sleep(UDP_SERVER_RESPAWN);
    } catch (InterruptedException e) {
    e.printStackTrace();
    }
    } while (keepRunning);

    if (socket != null) {
    socket.close();
    }
    }

    private Runnable updateTelemetry = new Runnable() {
    public void run() {
    if (gcsUDPReceiver == null) return;

    final String message = gcsUDPReceiver.messagesQueue.poll();

    if (message == null) return;

    telemetryTask(message);
    }
    };


    I thought that the outer do...while loop in UDPReceiver should fix the problem respawning the dead connection but this's not working as I expected, moreover setReuseAddress(true) directive seems to be a source of issues instead a partial fix.



    I'm probably doing some other error but I'm not able to catch it.










    share|improve this question



























      0












      0








      0







      I'm writing an Android application (API 24) that should receive telemetry messages trough an UDP connection. It works fine if the (wireless) connection is in good state, as long as it degrades the UDP receiver stops working and I'm not able to respawn even when the connection it's again in good state.



      This's my code:



      private class UDPReceiver extends Thread {
      private boolean keepRunning = true;
      private int UDP_SERVER_PORT;
      public Queue<String> messagesQueue = new LinkedList<>();

      public void run() {
      byte buffer = new byte[MAX_UDP_DATAGRAM_LEN];

      DatagramPacket packet = new DatagramPacket(buffer, buffer.length);
      DatagramSocket socket = null;

      do {
      try {
      socket = new DatagramSocket(UDP_SERVER_PORT);
      socket.setReuseAddress(true);

      while (keepRunning) {
      socket.receive(packet);
      messagesQueue.add(new String(buffer, 0, packet.getLength()));
      runOnUiThread(updateTelemetry);
      }
      } catch (Throwable e) {
      Log.e(this.getClass().getName(), e.getMessage());
      e.printStackTrace();
      }

      try {
      Thread.sleep(UDP_SERVER_RESPAWN);
      } catch (InterruptedException e) {
      e.printStackTrace();
      }
      } while (keepRunning);

      if (socket != null) {
      socket.close();
      }
      }

      private Runnable updateTelemetry = new Runnable() {
      public void run() {
      if (gcsUDPReceiver == null) return;

      final String message = gcsUDPReceiver.messagesQueue.poll();

      if (message == null) return;

      telemetryTask(message);
      }
      };


      I thought that the outer do...while loop in UDPReceiver should fix the problem respawning the dead connection but this's not working as I expected, moreover setReuseAddress(true) directive seems to be a source of issues instead a partial fix.



      I'm probably doing some other error but I'm not able to catch it.










      share|improve this question















      I'm writing an Android application (API 24) that should receive telemetry messages trough an UDP connection. It works fine if the (wireless) connection is in good state, as long as it degrades the UDP receiver stops working and I'm not able to respawn even when the connection it's again in good state.



      This's my code:



      private class UDPReceiver extends Thread {
      private boolean keepRunning = true;
      private int UDP_SERVER_PORT;
      public Queue<String> messagesQueue = new LinkedList<>();

      public void run() {
      byte buffer = new byte[MAX_UDP_DATAGRAM_LEN];

      DatagramPacket packet = new DatagramPacket(buffer, buffer.length);
      DatagramSocket socket = null;

      do {
      try {
      socket = new DatagramSocket(UDP_SERVER_PORT);
      socket.setReuseAddress(true);

      while (keepRunning) {
      socket.receive(packet);
      messagesQueue.add(new String(buffer, 0, packet.getLength()));
      runOnUiThread(updateTelemetry);
      }
      } catch (Throwable e) {
      Log.e(this.getClass().getName(), e.getMessage());
      e.printStackTrace();
      }

      try {
      Thread.sleep(UDP_SERVER_RESPAWN);
      } catch (InterruptedException e) {
      e.printStackTrace();
      }
      } while (keepRunning);

      if (socket != null) {
      socket.close();
      }
      }

      private Runnable updateTelemetry = new Runnable() {
      public void run() {
      if (gcsUDPReceiver == null) return;

      final String message = gcsUDPReceiver.messagesQueue.poll();

      if (message == null) return;

      telemetryTask(message);
      }
      };


      I thought that the outer do...while loop in UDPReceiver should fix the problem respawning the dead connection but this's not working as I expected, moreover setReuseAddress(true) directive seems to be a source of issues instead a partial fix.



      I'm probably doing some other error but I'm not able to catch it.







      android udp






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Nov 15 '18 at 7:23

























      asked Nov 14 '18 at 9:59









      weirdgyn

      279326




      279326
























          0






          active

          oldest

          votes











          Your Answer






          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "1"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: true,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: 10,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });














          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53297451%2fudp-server-respawn-after-timeout%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes
















          draft saved

          draft discarded




















































          Thanks for contributing an answer to Stack Overflow!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.





          Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


          Please pay close attention to the following guidance:


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53297451%2fudp-server-respawn-after-timeout%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          Guess what letter conforming each word

          Port of Spain

          Run scheduled task as local user group (not BUILTIN)