Skip to content
This repository was archived by the owner on Feb 11, 2025. It is now read-only.

Commit 60d455b

Browse files
authored
Merge pull request #55 from fullstack-devops/bugfix/fix-java-cert-import
fix: java cert import > v1.8 + update actions
2 parents 324fae6 + 412f7e8 commit 60d455b

File tree

4 files changed

+20
-8
lines changed

4 files changed

+20
-8
lines changed

.github/workflows/build-pr.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
uses: docker/setup-buildx-action@v2
4949

5050
- name: Build github-actions-runner:base
51-
uses: docker/build-push-action@v3
51+
uses: docker/build-push-action@v4
5252
with:
5353
context: ./images/base
5454
push: true
@@ -83,9 +83,9 @@ jobs:
8383
images/${{ matrix.flavour }}/Dockerfile
8484
8585
- name: Build github-actions-runner:${{ matrix.flavour }}
86-
uses: docker/build-push-action@v3
86+
uses: docker/build-push-action@v4
8787
with:
8888
context: ./images/${{ matrix.flavour }}
89-
push: true
89+
push: false
9090
tags: |
9191
${{ env.IMAGE_BASE }}:${{ matrix.flavour }}-pr-${{ github.event.pull_request.number }}

.github/workflows/create-release.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ jobs:
5959
uses: docker/setup-buildx-action@v2
6060

6161
- name: Build github-actions-runner:base
62-
uses: docker/build-push-action@v3
62+
uses: docker/build-push-action@v4
6363
with:
6464
context: ./images/base
6565
push: true
@@ -98,7 +98,7 @@ jobs:
9898
images/${{ matrix.flavour }}/Dockerfile
9999
100100
- name: Build github-actions-runner:${{ matrix.flavour }}
101-
uses: docker/build-push-action@v3
101+
uses: docker/build-push-action@v4
102102
with:
103103
context: ./images/${{ matrix.flavour }}
104104
push: true

.vscode/extensions.json

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"recommendations": [
3+
"shakram02.bash-beautify"
4+
]
5+
}

images/base/helper-scripts/detect-setup.sh

100644100755
+10-3
Original file line numberDiff line numberDiff line change
@@ -17,18 +17,25 @@ if command -v java --version &> /dev/null; then
1717
echo "generating single certs at $javacerts/"
1818
mkdir -p $javacerts
1919
cat $CA_FILE | awk 'split_after==1{n++;split_after=0} /-----END CERTIFICATE-----/ {split_after=1} {print > ("/tmp/javacerts/cert" n ".crt")}'
20-
20+
2121
for f in $javacerts/*.crt ; do
2222
# delete empty lines in files
2323
sed -i '/^$/d' $f
2424
done
2525
# delete empty files to prevent errors at import
2626
find $javacerts -empty -delete
27-
27+
28+
java_version=$(java -version 2>&1 | awk -F '"' '/version/ {print $2}')
2829
for x in $javacerts/*.crt ; do
2930
alias=$(openssl x509 -noout -subject -in "$x" | awk -F= '{print $NF}' | sed -e 's/^[ \t]*//' | sed -e 's/ /_/g')
3031
echo "importing cert of $alias"
31-
keytool -importcert -alias $alias -keystore /usr/lib/jvm/adoptopenjdk-8-hotspot-amd64/jre/lib/security/cacerts -storepass changeit -file $x -noprompt
32+
if [[ "$java_version" > "1.8" ]]; then
33+
echo "using java greater 1.8"
34+
keytool -importcert -alias $alias -cacerts -storepass changeit -file $x -noprompt
35+
else
36+
echo "using java lower 1.8"
37+
keytool -importcert -alias $alias -keystore /usr/lib/jvm/adopt*/jre/lib/security/cacerts -storepass changeit -file $x -noprompt
38+
fi
3239
done
3340
rm -rf $javacerts
3441
fi

0 commit comments

Comments
 (0)