Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add runcodeql.sh for "tox -e codeql" - Security check in python codes #105

Merged
merged 3 commits into from
Jan 26, 2023
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion src/tox_lsr/config_files/tox-default.ini
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ envlist =
black, pylint, flake8, yamllint
py{26,27,36,37,38,39,310,311}, shellcheck
collection, ansible-lint, custom
ansible-test, woke
ansible-test, woke, codeql
skipsdist = true
skip_missing_interpreters = true

Expand Down Expand Up @@ -272,6 +272,11 @@ changedir = {toxinidir}
commands =
bash {lsr_scriptdir}/runwoke.sh

[testenv:codeql]
changedir = {toxinidir}
commands =
bash {lsr_scriptdir}/runcodeql.sh

[qemu_common]
changedir = {toxinidir}
basepython = python3
Expand Down
116 changes: 116 additions & 0 deletions src/tox_lsr/test_scripts/runcodeql.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,116 @@
#!/bin/bash
# SPDX-License-Identifier: MIT

# Do not exit on an error to continue ansible-doc and ansible-test.
set -euo pipefail

#uncomment if you use $ME - otherwise set in utils.sh
#ME=$(basename "$0")
SCRIPTDIR=$(readlink -f "$(dirname "$0")")

. "${SCRIPTDIR}/utils.sh"

# Run codeql against python codes in a role
CODEQLACTIONDIR=${CODEQLACTIONDIR:-"${HOME}/github.com/github/codeql-action"}
ROLE=${ROLE:-"$( basename $TOPDIR )"}
JQPATH=$( which jq 2> /dev/null )
if [ $? -ne 0 ]; then
lsr_error "${ME}: jq is missing. Please install the package."
fi

# Go to the TOPDIR
cd "$TOPDIR"

# Install CodeQL
# https://docs.github.com/en/code-security/code-scanning/using-codeql-code-scanning-with-your-existing-ci-system/installing-codeql-cli-in-your-ci-system
CODEQLTARBALL=codeql-bundle-linux64.tar.gz
CODEQLURL=https://github.com/github/codeql-action/releases/latest/download/$CODEQLTARBALL
if [ ! -f "$LSR_TOX_ENV_TMP_DIR/$CODEQLTARBALL" ]; then
curl -L -o "$LSR_TOX_ENV_TMP_DIR/$CODEQLTARBALL" "$CODEQLURL"
fi
if [ ! -d "$LSR_TOX_ENV_TMP_DIR/codeql" ]; then
tar xfz "$LSR_TOX_ENV_TMP_DIR/$CODEQLTARBALL" -C "$LSR_TOX_ENV_TMP_DIR"
fi
# codeql/codeql is a shell script which launches java, which requires all the files in the
PATH="$LSR_TOX_ENV_TMP_DIR/codeql":"$PATH"

# Checkout codeql-action
CODEQLACTIONDIR="$LSR_TOX_ENV_DIR/codeql-action"
if [ ! -d "$CODEQLACTIONDIR" ]; then
git clone https://github.com/github/codeql-action "$CODEQLACTIONDIR"
fi

# Create a database dir:
DBDIR=$LSR_TOX_ENV_DIR/database
if [ ! -d "$DBDIR" ]; then
mkdir $DBDIR
fi
RESULTS=$LSR_TOX_ENV_DIR/results
if [ ! -d "$RESULTS" ]; then
mkdir $RESULTS
fi

# Load language configuration
codeql resolve queries python-code-scanning.qls --format=bylanguage

codeql resolve queries python-security-and-quality.qls --format=bylanguage

codeql resolve languages --format=betterjson --extractor-options-verbosity=4

# Setup Python dependencies
# $CODEQLACTIONDIR/python-setup/install_tools.sh
# Remove "--user" from "pip install" to workaround this error.
# ERROR: Can not perform a '--user' install. User site-packages are
# not visible in this virtualenv.
sed -e "s/pip install --user/pip install/" \
$CODEQLACTIONDIR/python-setup/install_tools.sh > "$LSR_TOX_ENV_TMP_DIR/install_tools.sh"
bash "$LSR_TOX_ENV_TMP_DIR/install_tools.sh"

codeql database init --db-cluster "$DBDIR" --source-root="$TOPDIR" \
--language=python

# Setup environment variables
export CODEQL_WORKFLOW_STARTED_AT=$( date -Iseconds )
export CODEQL_RAM=5919
export CODEQL_THREADS=2

# Extracting python
codeql database trace-command "$DBDIR/python" -- \
"$LSR_TOX_ENV_TMP_DIR/codeql/python/tools/autobuild.sh"

# Finalizing python
codeql database finalize --finalize-dataset --threads="$CODEQL_THREADS" \
--ram="$CODEQL_RAM" "$DBDIR/python"

# Running queries for python
codeql database run-queries --ram="$CODEQL_RAM" --threads="$CODEQL_THREADS" \
"$DBDIR/python" --min-disk-free=1024 \
-v python-security-and-quality.qls

# Interpreting results for python
codeql database interpret-results --threads="$CODEQL_THREADS" \
--format=sarif-latest -v --output=$RESULTS/python.sarif \
--no-sarif-add-snippets --print-diagnostics-summary \
--print-metrics-summary --sarif-group-rules-by-pack \
--sarif-add-query-help --sarif-category /language:python \
--sarif-add-baseline-file-info "$DBDIR/python" \
python-security-and-quality.qls

codeql database print-baseline "$DBDIR/python"

echo "CodeQL result file on $ROLE: $RESULTS/python.sarif"

JQPATH=$( which jq 2> /dev/null )
if [ $? -ne 0 ]; then
echo "WARNING: please install jq package"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move this check to near the top so the script fails early if jq not found

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove this check since you added it to the top

else
rcnt=$( jq '.runs[0].results | length' "$RESULTS/python.sarif" )
if [ $rcnt -gt 0 ]; then
echo "CODEQL RESULT"
jq '.runs[0].results' "$RESULTS/python.sarif"
lsr_error "${ME}: Found $rcnt security and quality issue(s)."
else
echo "PASS: Found no security and quality issues."
fi
fi
exit 0
4 changes: 4 additions & 0 deletions tests/fixtures/test_tox_merge_ini/result.ini
Original file line number Diff line number Diff line change
Expand Up @@ -226,6 +226,10 @@ commands = bash {lsr_scriptdir}/runansible-test.sh
changedir = {toxinidir}
commands = bash {lsr_scriptdir}/runwoke.sh

[testenv:codql]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[testenv:codql]
[testenv:codeql]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shame shame Thanks, @richm!

changedir = {toxinidir}
commands = bash {lsr_scriptdir}/runcodql.sh

[qemu_common]
changedir = {toxinidir}
basepython = python3
Expand Down