Skip to content

Commit 44e84b0

Browse files
authored
Merge pull request #189 from companieshouse/feature/rollover-dependency-check
Updated Makefile for dependency
2 parents e6136e7 + 53f279a commit 44e84b0

File tree

3 files changed

+4
-47
lines changed

3 files changed

+4
-47
lines changed

Makefile

Lines changed: 3 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,6 @@
11
artifact_name := psc-data-api
22
version := "unversioned"
33

4-
dependency_check_base_suppressions:=common_suppressions_spring_6.xml
5-
dependency_check_suppressions_repo_branch:=main
6-
dependency_check_minimum_cvss := 4
7-
dependency_check_assembly_analyzer_enabled := false
8-
dependency_check_suppressions_repo_url:[email protected]:companieshouse/dependency-check-suppressions.git
9-
suppressions_file := target/suppressions.xml
10-
11-
124
.PHONY: all
135
all: build
146

@@ -72,40 +64,8 @@ publish:
7264

7365
.PHONY: sonar
7466
sonar:
75-
mvn sonar:sonar -Dsonar.dependencyCheck.htmlReportPath=./target/dependency-check-report.html
67+
mvn sonar:sonar
7668

7769
.PHONY: sonar-pr-analysis
78-
sonar-pr-analysis: dependency-check
79-
mvn sonar:sonar -P sonar-pr-analysis -Dsonar.dependencyCheck.htmlReportPath=./target/dependency-check-report.html
80-
81-
.PHONY: dependency-check
82-
dependency-check:
83-
@ if [ -d "$(DEPENDENCY_CHECK_SUPPRESSIONS_HOME)" ]; then \
84-
suppressions_home="$${DEPENDENCY_CHECK_SUPPRESSIONS_HOME}"; \
85-
fi; \
86-
if [ ! -d "$${suppressions_home}" ]; then \
87-
suppressions_home_target_dir="./target/dependency-check-suppressions"; \
88-
if [ -d "$${suppressions_home_target_dir}" ]; then \
89-
suppressions_home="$${suppressions_home_target_dir}"; \
90-
else \
91-
mkdir -p "./target"; \
92-
git clone $(dependency_check_suppressions_repo_url) "$${suppressions_home_target_dir}" && \
93-
suppressions_home="$${suppressions_home_target_dir}"; \
94-
if [ -d "$${suppressions_home_target_dir}" ] && [ -n "$(dependency_check_suppressions_repo_branch)" ]; then \
95-
cd "$${suppressions_home}"; \
96-
git checkout $(dependency_check_suppressions_repo_branch); \
97-
cd -; \
98-
fi; \
99-
fi; \
100-
fi; \
101-
suppressions_path="$${suppressions_home}/suppressions/$(dependency_check_base_suppressions)"; \
102-
if [ -f "$${suppressions_path}" ]; then \
103-
cp -av "$${suppressions_path}" $(suppressions_file); \
104-
mvn org.owasp:dependency-check-maven:check -Dformats="json,html" -DprettyPrint -DfailBuildOnCVSS=$(dependency_check_minimum_cvss) -DassemblyAnalyzerEnabled=$(dependency_check_assembly_analyzer_enabled) -DsuppressionFiles=$(suppressions_file); \
105-
else \
106-
printf -- "\n ERROR Cannot find suppressions file at '%s'\n" "$${suppressions_path}" >&2; \
107-
exit 1; \
108-
fi
109-
110-
.PHONY: security-check
111-
security-check: dependency-check
70+
sonar-pr-analysis:
71+
mvn sonar:sonar -P sonar-pr-analysis

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>uk.gov.companieshouse</groupId>
77
<artifactId>companies-house-parent</artifactId>
8-
<version>2.1.11</version>
8+
<version>2.1.12</version>
99
</parent>
1010
<artifactId>psc-data-api</artifactId>
1111
<version>unversioned</version>

suppress.xml

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)