Always use our async DNS in callback interface

Our own async DNS has been "upgraded to full membership". It's now
used always in the callback interface on POSIX and Windows and
OpenSSL. This avoids the ugly blocking behavior of getaddrinfo(),
which is most ugly when there's trouble w/networking.

A few more samples were added, illustrating common usage patterns,
though they are somewhat "initial", will be improved with time.

Also, most samples are fixed to make nice w/HTTP keep-alive.
Functional tests ditto (though it seems there's some more work
needed there, especially on OpenSSL).

Author: vveljko

.pubnub.yml

@@ -1,8 +1,10 @@
 name: c-core
-version: 2.3.4
+version: 2.4.0
 scm: github.com/pubnub/c-core
 changelog:
-  - version: v2.3.4
+  - version: v2.4.0
+      - type: feature
+        text: Callback interface uses (our own) async DNS
       - type: feature
         text: Enable sending meta data when publishing
       - type: feature

VERSION

@@ -1 +1 @@
-2.3.4
+2.4.0

core/fntest/pubnub_fntest.c

@@ -3,8 +3,10 @@
 #include "core/pubnub_coreapi.h"
 #include "core/pubnub_assert.h"
 #include "pubnub_internal.h"
+#include "core/pubnub_ntf_sync.h"
+#include "core/pubnub_alloc.h"
 
-#include "core/fntest/pubnub_fntest.h"
+#include "pubnub_fntest.h"
 
 #include <string.h>
 #include <stdarg.h>
@@ -109,10 +111,7 @@ bool pnfntst_subscribe_and_check(pubnub_t *p, char const *channel, char const*ch
     pnfntst_start_timer(tmr, ms);
     while (pnfntst_timer_is_running(tmr) && missing) {
         enum pubnub_res pbres = pubnub_subscribe(p, channel, chgroup);
-        if (PNR_STARTED != pbres) {
-            puts("subscribe and check: subscribe failed");
-            break;
-        }
+        if (PNR_STARTED == pbres) {
         while (pnfntst_timer_is_running(tmr)) {
             pbres = pubnub_last_result(p);
             if (pbres != PNR_STARTED) {
@@ -123,7 +122,10 @@ bool pnfntst_subscribe_and_check(pubnub_t *p, char const *channel, char const*ch
             printf("subscribe and check: subscribe error %d\n", pbres);
             break;
         }
-	
+        } else if (pbres != PNR_OK) {
+            puts("subscribe and check: subscribe failed");
+            break;
+        }
         for (;;) {
             size_t i;
             char const *msg = pubnub_get(p);
@@ -146,3 +148,12 @@ bool pnfntst_subscribe_and_check(pubnub_t *p, char const *channel, char const*ch
 
     return !missing;
 }
+
+void pnfntst_free(pubnub_t *p)
+{
+    pubnub_cancel(p);
+    pubnub_await(p);
+    if (pubnub_free(p) != 0) {
+        printf("Failed to free the Pubnub context\n");
+    }
+}

core/fntest/pubnub_fntest.h

@@ -51,6 +51,7 @@ bool pnfntst_got_message_on_channel(pubnub_t *p, char const *message, char const
 
 
 bool pnfntst_subscribe_and_check(pubnub_t *p, char const *chan, char const*chgroup, unsigned ms, ...);
+void pnfntst_free(pubnub_t *p);
 
 #include "fntest/pubnub_fntest_pal.h"