feat(docker): Improve docker build and prevent double path in naming

Signed-off-by: Helio Chissini de Castro <dev@heliocastro.info>

Author: heliocastro

.github/workflows/docker_deploy.yml

@@ -6,9 +6,14 @@
 name: Docker Build
 
 on:
+  workflow_dispatch:
+  pull_request:
+    paths:
+      - Dockerfile
+      - .github/workflows/docker_deploy.yml
   push:
     paths-ignore:
-      - "**.md"
+      - '**.md'
     tags:
       - 'v*'
 env:
@@ -24,42 +29,34 @@ jobs:
 
     steps:
       - name: Checkout main repository
-        uses: actions/checkout@v3
-
-      - name: Set environment variables
-        run: |
-          echo "ORG_BASE_NAME=${GITHUB_REPOSITORY}" >> $GITHUB_ENV
-          echo "TOOLS_JAVA_VERSION=${GITHUB_REF_NAME/v/}" >> $GITHUB_ENV
-      - name: Echoing current version
-        run: |
-          echo "$TOOLS_JAVA_VERSION"
-          echo $GITHUB_REF_NAME
+        uses: actions/checkout@v5
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v2
+        uses: docker/setup-buildx-action@8d2750c68a42422c14e847fe6c8ac0403b4cbd6f # v3.12.0
 
       - name: Login to GitHub Container Registry
-        uses: docker/login-action@v2
+        uses: docker/login-action@5e57cd118135c172c3672efd75eb46360885c0ef # v3.6.0
         with:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Extract components metadata
         id: meta_base
-        uses: docker/metadata-action@v4
+        uses: docker/metadata-action@c299e40c65443455700f0fdfc63efafe5b349051 # v51.0.0
         with:
-          images: |
-            ${{ env.REGISTRY }}/${{ env.ORG_BASE_NAME }}/tools-java
+          tags: |
+            type=ref,event=tag
+            type=semver,pattern={{version}}
+            type=raw,value=main,enable=${{ github.ref == 'refs/heads/main' }}
+          labels: org.opencontainers.image.licenses=Apache-2.0
 
       - name: Build Container
-        uses: docker/build-push-action@v3
+        uses: docker/build-push-action@263435318d21b8e681c14492fe198d362a7d2c83 # v6.18.0
         with:
           context: .
-          push: true
+          push: ${{ github.event_name == 'push' && startsWith(github.ref, 'refs/tags/') }}
           load: false
-          build-args: |
-            TOOLS_JAVA_VERSION=${{ env.TOOLS_JAVA_VERSION }}
           tags: |
             ${{ steps.meta_base.outputs.tags }}
           labels: ${{ steps.meta_base.outputs.labels }}

Dockerfile

@@ -1,35 +1,32 @@
 # syntax=docker/dockerfile:1.4
 
 # Set Java versions
-ARG JAVA_VERSION=17
+ARG JAVA_VERSION=21
 
 # Use Maven eclipse Temurin based
-FROM maven:3.8-eclipse-temurin-$JAVA_VERSION as build
-
-ARG TOOLS_JAVA_VERSION=1.1.5-SNAPSHOT
+FROM maven:3.9-eclipse-temurin-$JAVA_VERSION as build
 
 WORKDIR /build
 
 # BUILD
 RUN --mount=type=cache,target=/root/.m2 \
     --mount=type=bind,source=$PWD,target=/build,rw \
-    mvn clean org.jacoco:jacoco-maven-plugin:prepare-agent install \
+    export TOOLS_JAVA_VERSION=$(mvn help:evaluate -Dexpression=project.version -q -DforceStdout) \
+    && mvn clean org.jacoco:jacoco-maven-plugin:prepare-agent install \
     && mkdir -p /usr/lib/java/spdx \
     && cp target/tools-java-$TOOLS_JAVA_VERSION-jar-with-dependencies.jar /usr/lib/java/spdx/
 
-
 # Configure the wrapper script
 COPY scripts/tools-java-wrapper.sh /usr/bin/tools-java
 
 # Make the bwrapper match tools version
 RUN sed -i "s/@@VERSION@@/$TOOLS_JAVA_VERSION/g" /usr/bin/tools-java \
     && chmod +x /usr/bin/tools-java
 
-
 # Deploy image
 FROM eclipse-temurin:$JAVA_VERSION as run
 
 COPY --from=build /usr/lib/java/spdx /usr/lib/java/spdx
 COPY --from=build /usr/bin/tools-java /usr/bin/tools-java
 
-ENTRYPOINT [ "/usr/bin/tools-java" ]
+ENTRYPOINT [ "/usr/bin/tools-java" ]