Fix upstream missing USE_TLS macro guard (#242)

* Fix upstream missing USE_TLS macro guard

* Added --disable-tls build variation to prevent this issue in the future

---------

Co-authored-by: filipecosta90 <[email protected]>

Author: Alkaid-Benetnash

.github/workflows/ci.yml

@@ -3,6 +3,18 @@ name: CI
 on: [push, pull_request]
 
 jobs:
+  build-notls:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - name: Install dependencies
+      run: |
+        sudo apt-get -qq update
+        sudo apt-get install lcov autoconf automake pkg-config libevent-dev libpcre3-dev
+
+    - name: Build
+      run: autoreconf -ivf && ./configure --disable-tls && make -j
+
   build-ubuntu:
     strategy:
       matrix:

memtier_benchmark.cpp

@@ -313,8 +313,10 @@ static void config_init_defaults(struct benchmark_config *cfg)
         cfg->hdr_prefix = "";
     if (!cfg->print_percentiles.is_defined())
         cfg->print_percentiles = config_quantiles("50,99,99.9");
+#ifdef USE_TLS
     if (!cfg->tls_protocols)
         cfg->tls_protocols = REDIS_TLS_PROTO_DEFAULT;
+#endif
 }
 
 static int generate_random_seed()