cerbero ci: Don't expand the ssh private key
authorNirbheek Chauhan <nirbheek@centricular.com>
Thu, 17 Sep 2020 16:21:49 +0000 (21:51 +0530)
committerNirbheek Chauhan <nirbheek@centricular.com>
Thu, 17 Sep 2020 17:48:30 +0000 (23:18 +0530)
Because of set -x we were printing the key into the CI logs:
https://gitlab.freedesktop.org/gstreamer/cerbero/-/jobs/4572836#L797

Don't ever expand the variable in shell.

gitlab/cerbero_setup.sh

index c4e5251..48a8a28 100644 (file)
@@ -107,7 +107,10 @@ cerbero_deps_script() {
         rsync -aH "${CERBERO_OVERRIDDEN_DIST_DIR}/" "${CERBERO_HOME}/dist/${ARCH}"
     fi
 
-    if [[ -n ${CERBERO_PRIVATE_SSH_KEY} ]]; then
+    # Check that the env var is set. Don't expand this protected variable by
+    # doing something silly like [[ -n ${CERBERO_...} ]] because it will get
+    # printed in the CI logs due to set -x
+    if env | grep -q -e CERBERO_PRIVATE_SSH_KEY; then
         $CERBERO $CERBERO_ARGS gen-cache --branch "${GST_UPSTREAM_BRANCH}"
         $CERBERO $CERBERO_ARGS upload-cache --branch "${GST_UPSTREAM_BRANCH}"
     fi