Address codacy issues

CHANGELOG.md

@@ -4,9 +4,9 @@
 
 ## MINOR Feature
 1. [912] - Adds initial compose file and docker files for python-client
-1. [909] - Added Support for Google Analytics
-2. [916] - Reworked Docker build process to be faster and produce simpler and smaller images
-2. [912] - Adding working compose instance for core metadata services.
+2. [909] - Added Support for Google Analytics
+3. [916] - Reworked Docker build process to be faster and produce simpler and smaller images
+4. [912] - Adding working compose instance for core metadata services.
 
 ## PATCH Bug fixes/Technical Debt/Documentation
 1. [914] - Improve GitHub template

compose/build_images_for_compose.sh

@@ -4,7 +4,7 @@ set -euf -o pipefail
 
 SCRIPT=$(realpath "$0")
 SOURCE=$(dirname "$SCRIPT")
-PROJECT_ROOT=$(realpath ${SOURCE}/../)
+PROJECT_ROOT=$(realpath "${SOURCE}/../")
 
 docker build \
   -f "${PROJECT_ROOT}/docker/Dockerfile.dependencies" \

compose/compose.yml

@@ -2,6 +2,15 @@ version: '3.9'
 
 
 services:
+
+  keycloak:
+    image: jboss/keycloak:latest
+    environment:
+      KEYCLOAK_USER: admin
+      KEYCLOAK_PASSWORD: admin
+    ports:
+      - "8888:8080"
+
   datafed-web:
     depends_on: ["datafed-core"]
     environment:

docker/entrypoint_foxx.sh

@@ -8,7 +8,7 @@ fi
 
 SCRIPT=$(realpath "$0")
 SOURCE=$(dirname "$SCRIPT")
-PROJECT_ROOT=$(realpath ${SOURCE}/../)
+PROJECT_ROOT=$(realpath "${SOURCE}/../")
 
 log_path="$DATAFED_DEFAULT_LOG_PATH"
 if [ ! -d "${log_path}" ]

scripts/export_dependency_version.sh

@@ -3,7 +3,7 @@
 # Assumes sourcing not running
 SCRIPT=$( realpath "${BASH_SOURCE[0]}" )
 SCRIPT_DIR=$( dirname "${SCRIPT}" )
-PROJECT_ROOT=$(realpath ${SCRIPT_DIR}/..)
+PROJECT_ROOT=$(realpath "${SCRIPT_DIR}/..")
 
 echo "SCRIPT $SCRIPT"
 echo "SCRIPT_DIR $SCRIPT_DIR"

scripts/install_lego_and_certificates.sh

@@ -4,9 +4,9 @@ set -euf -o pipefail
 
 SCRIPT=$(realpath "$0")
 SOURCE=$(dirname "$SCRIPT")
-PROJECT_ROOT=$(realpath ${SOURCE}/..)
-source ${PROJECT_ROOT}/config/datafed.sh
-source ${PROJECT_ROOT}/scripts/utils.sh
+PROJECT_ROOT=$(realpath "${SOURCE}/..")
+source "${PROJECT_ROOT}/config/datafed.sh"
+source "${PROJECT_ROOT}/scripts/utils.sh"
 
 Help()
 {

scripts/install_ws.sh

@@ -4,39 +4,38 @@ set -uf -o pipefail
 
 SCRIPT=$(realpath "$0")
 SOURCE=$(dirname "$SCRIPT")
-PROJECT_ROOT=$(realpath ${SOURCE}/..)
-source ${PROJECT_ROOT}/config/datafed.sh
-source ${SOURCE}/dependency_versions.sh
-source ${SOURCE}/dependency_install_functions.sh
+PROJECT_ROOT=$(realpath "${SOURCE}/..")
+source "${PROJECT_ROOT}/config/datafed.sh"
+source "${SOURCE}/dependency_versions.sh"
+source "${SOURCE}/dependency_install_functions.sh"
 
 #NVM_DIR=/home/cades/.nvm
 #[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"
 #export NVM_DIR="/home/cades/.nvm"
 #source ${NVM_DIR}/nvm.sh
 # Make sure paths exist
-mkdir -p ${DATAFED_INSTALL_PATH}/web
-mkdir -p ${DATAFED_INSTALL_PATH}/keys
-mkdir -p ${DATAFED_DEFAULT_LOG_PATH}
+mkdir -p "${DATAFED_INSTALL_PATH}/web"
+mkdir -p "${DATAFED_INSTALL_PATH}/keys"
+mkdir -p "${DATAFED_DEFAULT_LOG_PATH}"
 
 # Install web node modules
-cp "$PROJECT_ROOT/web/package.json" ${DATAFED_INSTALL_PATH}/web/
-
+cp "$PROJECT_ROOT/web/package.json" "${DATAFED_INSTALL_PATH}/web/"
 install_nvm
 install_node
 install_foxx_cli
 {
-  npm --allow-root --unsafe-perm --prefix ${DATAFED_INSTALL_PATH}/web install 
+  npm --allow-root --unsafe-perm --prefix "${DATAFED_INSTALL_PATH}/web" install 
 } || {
   echo "ERROR npm command failed!"
   exit 1
 }
 
 # Install javascript web server repo and core server were 
 # already installed by CMake
-cp "$PROJECT_ROOT/web/datafed-ws.js" ${DATAFED_INSTALL_PATH}/web
+cp "$PROJECT_ROOT/web/datafed-ws.js" "${DATAFED_INSTALL_PATH}/web"
 
 # Copy configuration files
-cp "$PROJECT_ROOT/config/datafed-ws.cfg" ${DATAFED_INSTALL_PATH}/web
+cp "$PROJECT_ROOT/config/datafed-ws.cfg" "${DATAFED_INSTALL_PATH}/web"
 
 # Make sure that the datafed-core-pub.key exists in the web/static folder
 if [[ -z  "${DATAFED_INSTALL_PATH}/keys/datafed-core-key.pub" ]]
@@ -45,4 +44,4 @@ then
   exit 1
 fi
 
-cp ${DATAFED_INSTALL_PATH}/keys/datafed-core-key.pub "$DATAFED_INSTALL_PATH/web/static/"
+cp "${DATAFED_INSTALL_PATH}/keys/datafed-core-key.pub" "$DATAFED_INSTALL_PATH/web/static/"

scripts/utils.sh

@@ -4,7 +4,7 @@ SOURCE=$(dirname "$SCRIPT")
 
 if [ -z "${PROJECT_ROOT}" ]
 then
-  PROJECT_ROOT=$(realpath ${SOURCE}/..)
+  PROJECT_ROOT=$(realpath "${SOURCE}/..")
 fi
 
 echo "PROJECT ROOT $PROJECT_ROOT"

web/docker/entrypoint.sh

@@ -4,19 +4,19 @@
 set -euf -o pipefail
 
 if [ -n "$UID" ]; then
-    usermod -u $UID datafed
+    usermod -u "$UID" datafed
 fi
 
 
 SCRIPT=$(realpath "$0")
 SOURCE=$(dirname "$SCRIPT")
-PROJECT_ROOT=$(realpath ${SOURCE}/../..)
+PROJECT_ROOT=$(realpath "${SOURCE}/../..")
 
 
-${PROJECT_ROOT}/scripts/generate_datafed.sh
-${PROJECT_ROOT}/scripts/generate_ws_config.sh
-${PROJECT_ROOT}/scripts/install_ws.sh
-. ${PROJECT_ROOT}/scripts/export_dependency_version.sh
+"${PROJECT_ROOT}/scripts/generate_datafed.sh"
+"${PROJECT_ROOT}/scripts/generate_ws_config.sh"
+"${PROJECT_ROOT}/scripts/install_ws.sh"
+. "${PROJECT_ROOT}/scripts/export_dependency_version.sh"
 
 export NVM_DIR="${DATAFED_DEPENDENCIES_INSTALL_PATH}/nvm"
 export NODE_VERSION="$DATAFED_NODE_VERSION"
@@ -30,7 +30,7 @@ if [ ! -L "/usr/local/bin/node" ]; then
   ln -s "$NODE_PATH" /usr/local/bin/node
 fi
 # Send output to file as well as print to terminal
-log_path=$(grep "log-path" ${BUILD_DIR}/config/datafed-ws.cfg | cut -d "=" -f 2 | tr -d ' ')
+log_path=$(grep "log-path" "${BUILD_DIR}/config/datafed-ws.cfg" | cut -d "=" -f 2 | tr -d ' ')
 
 if [ ! -d "${log_path}" ]
 then