Merge pull request #275 from CosmWasm/wasmd-ports

Don't open wamd Tendermint and p2p ports

scripts/wasmd/start.sh

@@ -2,6 +2,8 @@
 set -o errexit -o nounset -o pipefail
 command -v shellcheck > /dev/null && shellcheck "$0"
 
+REST_PORT_GUEST="1317"
+REST_PORT_HOST="1317"
 SCRIPT_DIR="$(realpath "$(dirname "$0")")"
 # shellcheck source=./env
 # shellcheck disable=SC1091
@@ -16,12 +18,11 @@ WASMD_LOGFILE="$TMP_DIR/wasmd.log"
 REST_SERVER_LOGFILE="$TMP_DIR/rest-server.log"
 
 # This starts up wasmd
+# The Tendermint port (26657) and the p2p port (26656) are not exposed since we don't need for testing
 docker volume rm -f wasmd_data
 docker run --rm \
   --name "$CONTAINER_NAME" \
-  -p 1317:1317 \
-  -p 26657:26657 \
-  -p 26656:26656 \
+  -p "$REST_PORT_HOST":"$REST_PORT_GUEST" \
   --mount type=bind,source="$SCRIPT_DIR/template",target=/template \
   --mount type=volume,source=wasmd_data,target=/root \
   "$REPOSITORY:$VERSION" \
@@ -47,10 +48,10 @@ docker exec "$CONTAINER_NAME" \
   --node tcp://localhost:26657 \
   --trust-node \
   --cors \
-  --laddr tcp://0.0.0.0:1317 \
+  --laddr "tcp://0.0.0.0:$REST_PORT_GUEST" \
   > "$REST_SERVER_LOGFILE" &
 
-echo "rest server running on http://localhost:1317 and logging into $REST_SERVER_LOGFILE"
+echo "rest server running on http://localhost:$REST_PORT_HOST and logging into $REST_SERVER_LOGFILE"
 
 # Give REST server some time to come alive. No idea why this helps. Needed for CI.
 if [ -n "${CI:-}" ]; then