Issue 68

.github/workflows/test.yml

@@ -25,7 +25,7 @@ jobs:
     runs-on: ${{ matrix.os }}
 
     steps:
-      - uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
+      - uses: actions/checkout@v4
 
       - uses: ./
         with:
@@ -41,22 +41,45 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
+      - uses: actions/checkout@v4
 
       - uses: ./
 
       - run: mvn -V validate -Drequire.java=17 -Drequire.maven=3.9.6
         shell: bash
 
+  # regressions tests
+  test-issue-68:
+    name: Test for issue 68
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: ./
+        with:
+          settings-servers: |
+            [{
+              "id": "ossrh",
+              "username": "${{ vars.TEST_68_1 }}",
+              "password": "${{ vars.TEST_68_2 }}"
+            }]
+
+      - run: cat ~/.m2/settings.xml
+      - run: |
+          grep "<id>ossrh</id>" ~/.m2/settings.xml
+          grep "<username>username-68</username>" ~/.m2/settings.xml
+          grep "<password>password-68</password>" ~/.m2/settings.xml
+
   test-ok:
     name: Test OK
     runs-on: ubuntu-latest
-    needs: [ test, test-default ]
+    needs: [ test, test-default, test-issue-68 ]
     if: always()
 
     steps:
       - run: 'true'
-        if: join(needs.*.result, ',') == 'success,success'
+        if: join(needs.*.result, ',') == 'success,success,success'
 
       - run: 'false'
-        if: join(needs.*.result, ',') != 'success,success'
+        if: join(needs.*.result, ',') != 'success,success,success'

README.md

@@ -40,7 +40,7 @@ For default values you only need:
 | params            | destination  | default |
 |-------------------|--------------|---------|
 | java-version      | java-version | 17      |
-| java-distribution | distribution | zulu    |
+| java-distribution | distribution | temurin |
 
 ## cache
 
@@ -80,7 +80,6 @@ So we can use for action:
 | settings-mirrors           | mirrors           |
 | settings-properties        | properties        |
 | settings-sonatypeSnapshots | sonatypeSnapshots |
-| settings-proxies           | proxies           |
 
 # Testing against different Maven versions
 

action.yml

@@ -44,7 +44,7 @@ inputs:
 
   java-distribution:
     description: 'Java distribution'
-    default: 'zulu'
+    default: 'temurin'
     required: false
 
   # cache
@@ -74,31 +74,27 @@ inputs:
 
   # maven settings.xml
   settings-servers:
-    description: 'servers definition in json array, e.g.: [{"id": "serverId", "username": "username", "password": "password"}]'
+    description: 'servers definition in json array, eg: [{"id": "serverId", "username": "username", "password": "password"}]'
     required: false
 
   settings-mirrors:
-    description: 'mirrors definition in json array, e.g.: [{"id": "id", "name": "name", "mirrorOf": "mirrorOf", "url": "url"}]'
+    description: 'mirrors definition in json array, eg: [{"id": "id", "name": "name", "mirrorOf": "mirrorOf", "url": "url"}]'
     required: false
 
   settings-properties:
-    description: 'json array with properties, e.g.: [{"propertyName1": "propertyValue1"}, {"propertyName2": "propertyValue2"}]'
+    description: 'json array with properties, eg [{"propertyName1": "propertyValue1"}, {"propertyName2": "propertyValue2"}]'
     required: false
 
   settings-sonatypeSnapshots:
     description: 'add https://oss.sonatype.org/content/repositories/snapshots to repository list - true or false'
     required: false
 
-  settings-proxies:
-    description: 'proxies definition in json array, e.g.: [{"id": "http-proxy", "active": "true", "protocol": "http", "host": "host", "port": "port", "nonProxyHosts": "host1|host2"}]'
-    required: false
-
 runs:
   using: 'composite'
 
   steps:
 
-    - uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
+    - uses: actions/checkout@v4
       with:
         fetch-depth: '${{ inputs.checkout-fetch-depth }}'
         path: '${{ inputs.checkout-path }}'
@@ -113,7 +109,7 @@ runs:
         java-version: '${{ inputs.java-version }}'
         distribution: '${{ inputs.java-distribution }}'
 
-    - uses: actions/cache@0c45773b623bea8c8e75f6c82b208c3cf94ea4f9 # v4.0.2
+    - uses: actions/cache@v4
       if: inputs.cache-enabled == 'true'
       with:
         path: |
@@ -127,15 +123,14 @@ runs:
       shell: bash
       id: current-maven
 
-    - uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
+    - uses: stCarolas/setup-maven@v5
       if: inputs.maven-version != steps.current-maven.outputs.version
       with:
         maven-version: '${{ inputs.maven-version }}'
 
-    - uses: s4u/maven-settings-action@7802f6aec16c9098b4798ad1f1d8ac75198194bd # v3.0.0
+    - uses: s4u/maven-settings-action@v3.0.0
       with:
-        servers: '${{ inputs.settings-servers }}'
+        servers: '${{ inputs.settings-servers}}'
-        mirrors: '${{ inputs.settings-mirrors }}'
+        mirrors: '${{ inputs.settings-mirrors}}'
-        properties: '${{ inputs.settings-properties }}'
+        properties: '${{ inputs.settings-properties}}'
-        sonatypeSnapshots: '${{ inputs.settings-sonatypeSnapshots }}'
+        sonatypeSnapshots: '${{ inputs.settings-sonatypeSnapshots}}'
-        proxies: '${{ inputs.settings-proxies }}'